mAutoFocusFired should be set when it's actually fired
Categories
(Core :: DOM: UI Events & Focus Handling, enhancement)
Tracking
()
People
(Reporter: sefeng211, Assigned: sefeng211)
References
Details
Attachments
(1 obsolete file)
The current approach is fine because there's only one place that we can dispatch an autofocus event, and we have some guards here to prevent it from dispatching twice, which is good.
The problem is once we added a dialog focusing steps for dialog element, it's possible that the autofocus event gets executed by dialog element, then a autofocus event may gets fired twice.
|
Assignee | |
Comment 1•6 years ago
|
||
The current approach is fine because there's only one place
that we can dispatch an autofocus event, and we have some guards
here to prevent it from dispatching twice, which is good.
The problem is once we added a dialog focusing steps for dialog
element, it's possible that the autofocus event gets executed
by dialog element, then a autofocus event may gets fired twice.
So let's check again before it's actually fired.
|
||
Updated•6 years ago
|
|
|
Assignee | |
Comment 2•6 years ago
|
||
Going to the changes along with bug 1324958
|
|
Assignee | |
Updated•6 years ago
|
Description
•