Bug 8724

Summary: REGRESSION: onSelect handler doesn't work for text fields
Product: WebKit Reporter: Adele Peterson <adele>
Component: FormsAssignee: Adele Peterson <adele>
Status: RESOLVED FIXED    
Severity: Normal Keywords: InRadar, Regression
Priority: P1    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch for multiple selection bugs - listed in changelog adele: review+

Adele Peterson
Reported 2006-05-03 16:25:42 PDT
onSelect handler doesn't work for text fields. This is definitely a regression, but I'm not sure when it regressed. It doesn't appear to work in our latest shipped version (2.0.3). From Radar comments, it looks like it was working sometime in March.
Attachments
patch for multiple selection bugs - listed in changelog (33.50 KB, patch)
2006-06-19 23:43 PDT, Adele Peterson 		adele: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2006-05-04 20:27:07 PDT
The way this was supposed to work with the old text field implementation was the call from RenderLineEdit::selectionChanged(). That in turn is supposed to be called from -[KWQTextFieldController textViewDidChangeSelection:].
Alice Liu
Comment 2 2006-06-06 07:18:40 PDT
<rdar://problem/4575082>
Adele Peterson
Comment 3 2006-06-19 23:43:59 PDT
Created attachment 8928 [details] patch for multiple selection bugs - listed in changelog
Adele Peterson
Comment 4 2006-06-20 17:05:57 PDT
revision 14931
Adele Peterson
Comment 5 2006-06-20 18:02:35 PDT
Comment on attachment 8928 [details] patch for multiple selection bugs - listed in changelog maciej reviewed this.
Note You need to log in before you can comment on or make changes to this bug.