Bug 6811

Summary: REGRESSION: Can't tab from old text field (like password fields) to new text field
Product: WebKit Reporter: Adele Peterson <adele>
Component: FormsAssignee: Maciej Stachowiak <mjs>
Status: RESOLVED FIXED    
Severity: Normal CC: c.petersen87
Priority: P1 Keywords: InRadar
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
test case
none
the fix is not so bad after all darin: review+

Adele Peterson
Reported 2006-01-25 18:30:48 PST
I'm not sure this matters that much, since the old text field code will soon go away. Anyway, this is probably a simple fix.
Attachments
test case (138 bytes, text/html)
2006-01-25 18:32 PST, Adele Peterson 		no flags
Details
the fix is not so bad after all (10.61 KB, patch)
2006-04-02 23:36 PDT, Maciej Stachowiak 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adele Peterson
Comment 1 2006-01-25 18:32:40 PST
Created attachment 5967 [details] test case test case added. Can't tab from the old field to the new field, but you can tab from the new to the old.
Adele Peterson
Comment 2 2006-03-01 16:57:20 PST
<rdar://problem/4463760> Can't tab from old text field to new text field (6811)
Adele Peterson
Comment 3 2006-03-29 17:49:06 PST
Bumping to P1 since this causes a text field regression
Maciej Stachowiak
Comment 4 2006-04-02 23:11:05 PDT
*** Bug 8122 has been marked as a duplicate of this bug. ***
Maciej Stachowiak
Comment 5 2006-04-02 23:25:35 PDT
These are all text field regressions so they should all be P1.
Maciej Stachowiak
Comment 6 2006-04-02 23:36:23 PDT
Created attachment 7479 [details] the fix is not so bad after all
Darin Adler
Comment 7 2006-04-03 08:41:38 PDT
Comment on attachment 7479 [details] the fix is not so bad after all I'm sad that nextKeyViewInFrame still needs to have a side effect. That having been said, this looks like a pretty good fix. r=me
Darin Adler
Comment 8 2006-04-03 08:44:01 PDT
Looks like Hyatt already reviewed this and Maciej already landed it last night.
Note You need to log in before you can comment on or make changes to this bug.