In the attached testcase: 1) Focus the first button using the keyboard 2) Press enter 3) Press TAB again and see that we now have two buttons with a focus ring
Simple patch with a test. Not sure that's the way to go, looking forward to your feedback!
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Attachment #601713 - Flags: review?(enndeakin)
Comment on attachment 601723 [details] [diff] [review] patch v2 This is ok, but a reftest would have been a simpler way of testing this.
Attachment #601723 - Flags: review?(enndeakin) → review+
(In reply to Neil Deakin from comment #4) > This is ok, but a reftest would have been a simpler way of testing this. Never wrote one, it was about time!
Comment on attachment 602420 [details] [diff] [review] patch v3 (with reftest) The line in reftest.list should be annotated with 'needs-focus' as the test depends on the window being focused.
Attachment #602420 - Flags: review?(enndeakin) → review+
Target Milestone: --- → mozilla13
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.