Bug 6811 - REGRESSION: Can't tab from old text field (like password fields) to new text field
: REGRESSION: Can't tab from old text field (like password fields) to new text ...
Status: RESOLVED FIXED
: WebKit
Forms
: 420+
: Macintosh Mac OS X 10.4
: P1 Normal
Assigned To:
:
: InRadar
:
:
  Show dependency treegraph
 
Reported: 2006-01-25 18:30 PST by
Modified: 2006-04-03 08:44 PST (History)


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 PST, Maciej Stachowiak
darin: review+
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 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.
------- Comment #1 From 2006-01-25 18:32:40 PST -------
Created an attachment (id=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.
------- Comment #2 From 2006-03-01 16:57:20 PST -------
<rdar://problem/4463760> Can't tab from old text field to new text field (6811)
------- Comment #3 From 2006-03-29 17:49:06 PST -------
Bumping to P1 since this causes a text field regression
------- Comment #4 From 2006-04-02 23:11:05 PST -------
*** Bug 8122 has been marked as a duplicate of this bug. ***
------- Comment #5 From 2006-04-02 23:25:35 PST -------
These are all text field regressions so they should all be P1.
------- Comment #6 From 2006-04-02 23:36:23 PST -------
Created an attachment (id=7479) [details]
the fix is not so bad after all
------- Comment #7 From 2006-04-03 08:41:38 PST -------
(From update of attachment 7479 [details])
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
------- Comment #8 From 2006-04-03 08:44:01 PST -------
Looks like Hyatt already reviewed this and Maciej already landed it last night.