12230 - In OPTGROUPs, fonts don't match when list dropped down

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect, P5)

Description

[cpratt]

Build ID: 1999082008
Platform: Windows NT (can't test on Mac OS yet)

To reproduce:
- Launch apprunner
- Load the above URL (Netscape internal only) or attached test case
- Click on the down arrow to drop down the list

Result: Although 12 point Times New Roman is used for items on the list, 10
point Arial is used for the currently selected option drawn above the top of the
list.

Expected result: The same font should be used if at all possible.

Comment 1

[karnaze (gone)]

Reassigning to Rod.

Comment 2

[rods (gone)]

Fixed.

Comment 3

[phillip]

apres moi, la deluge. (merci)

Comment 4

[cpratt]

Reopening and clearing resolution. Currently, we have:

Arial 10 pt - used when optionlist menu is not displayed
Times 12 pt bold italic - used for group headings
Arial 10 pt - used for selectable items in list

Is there a spec for this? I'm guessing that using Arial 10 pt for everything
*but* group headings is correct, but what to use for the group headings? I think
Arial 11 pt bold italic is where it's at, but I'm not part of the UI/UE team.
German? Shuang?

Comment 5

[cpratt]

Attachment: RTF file with a rough estimate of what this should look like

Comment 6

[cpratt]

Changing priority and severity (this is absolutely NOT something we need to look
at for beta!)...

Comment 7

[rods (gone)]

Fix in my tree

Comment 8

[rods (gone)]

They now have the same font - fixed

Comment 9

[cpratt]

Using the 1999101109 build under NT, the problem doesn't appear to be fixed. Is
this fixed in a later build or something? To verify the fix, download the
attachment and rename it 'foo.rtf'; open in Word, and compare it with the
optgroup generated by Mozilla. They don't currently match (you can see this as
the category headers have serifs, but the actual selectable items do not; try
comparing the 'a' in China with the 'a' in Yunnan).

Comment 10

[cpratt]

Perfect! 1999101908 build, NT.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a5a851cd7088
[components] Issue https://github.com/mozilla-mobile/android-components/issues/12230: Speculative fix for test failures in PromptFeatureTest