Closed
Bug 1106824
Opened 9 years ago
Closed 9 years ago
Focus does not change when click outside of select dropdown list
Categories
(Core :: DOM: UI Events & Focus Handling, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: alice0775, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: platform-parity, testcase, Whiteboard: [parity-Chrome][parity-IE])
Attachments
(2 files)
887 bytes,
text/html
|
Details | |
1.37 KB,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
Focus does not change when click outside of select dropdown. Steps To Reproduce: 1. Focus <select> element and Press Alt + down arrow key to popup dropdown 2. Click outside of the <select> element and the dropdown list Actual Results: The <select> element still has focus. In other words, onblur event does not fire on the <select> element. This causes Bug 1106812. Expected Results: Clicked element should get focus. I.e, onblur event should fire on the <select> element. The problem does not happen on IE11 and Chrome39.
Assignee | ||
Comment 1•9 years ago
|
||
> Expected Results: > Clicked element should get focus. Yes, I think that's expected on Windows, but not on Linux or OSX. It might be as simple as returning false in nsIRollupListener::Rollup... Alice, please test this build when it's ready: https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=2ff5ee883542
Flags: needinfo?(alice0775)
Assignee | ||
Updated•9 years ago
|
![]() |
Reporter | |
Comment 2•9 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #1) > > Expected Results: > > Clicked element should get focus. > > Yes, I think that's expected on Windows, but not on Linux or OSX. > > It might be as simple as returning false in nsIRollupListener::Rollup... > > Alice, please test this build when it's ready: > https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=2ff5ee883542 The try build works as expected for this testcase, and also works as expected for Bug 1106812. https://hg.mozilla.org/try/rev/2ff5ee883542 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:37.0) Gecko/20100101 Firefox/37.0 ID:20141202155041
Flags: needinfo?(alice0775)
Assignee | ||
Comment 3•9 years ago
|
||
There was a "layout/xul/test/test_bug987230.xul | Popup should only be shown once" failure on Try though. So let's try without the nsXULPopupManager change: https://tbpl.mozilla.org/?tree=Try&rev=b4e70d64964a
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8531703 -
Flags: review?(enndeakin)
Updated•9 years ago
|
Attachment #8531703 -
Flags: review?(enndeakin) → review+
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d37ced519d4d
Flags: in-testsuite?
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d37ced519d4d
Assignee: nobody → mats
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Updated•5 years ago
|
Component: Event Handling → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•