922082 - <select> with disabled and "display:none" option: Alpha-key selects hidden entry

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Markus Keller]

Attachment: test.html

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Firefox/24.0 (Beta/Release)
Build ID: 20130910160258

Steps to reproduce:

In a <select> list with disabled and invisible options (e.g. in the second list in the attachment), try to jump to an option by typing its first letter.


Actual results:

Focus jumps to before the first option.


Expected results:

Should jump to the first of the actually visible elements that matches the typed letter.

Comment 1

[Markus Keller]

<option>s with disabled attribute and style { display: none; } should not be considered when performing typing prefix completion.

In real life, this bug can be seen in Bugzilla 4.4, e.g. here, when you select the Product field and then type "s":
https://landfill.bugzilla.org/bugzilla-4.4-branch/query.cgi?classification=Unclassified
It works fine if you remove the "Classification" filter.

Bugzilla 4.2 used a different strategy to filter <select> lists, that's why this Firefox bug is not yet visible in the current bugzilla.mozilla.org. Works fine in Chrome.

Original report in Eclipse's Bugzilla:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=416542

Comment 2

[Markus Keller]

Use this Landfill link to ensure you're on the advanced search page: https://landfill.bugzilla.org/bugzilla-4.4-branch/query.cgi?format=advanced&classification=Unclassified

Comment 3

[Mats Palmgren (inactive)]

Attachment: fix

https://tbpl.mozilla.org/?tree=Try&rev=07e14e6bd426

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 812849 [details] [diff] [review]
fix

r=me

Comment 5

[Mats Palmgren (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8bc54ab0ef14

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/8bc54ab0ef14