129787 - [XBLFC] background of <select> popups is grey, not white (Classic/win2k).

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect)

Description

[John Morrison]

[XBLFC] background of <select> popups is grey, not white (Classic/win2k).

The background color of popups is grey, not white, in default style for Classic
on win2k.

Also, it appears that this cannot be overridden by author style rules, in 
either Modern or Classic. e.g.,

<html>
<head>
<style>
#foo { background-color: #ff0030; color: #00ff00; font-size : x-small;}
</style>
</head>
<body>
<form>
<select id="foo">
  <option value="no" selected>same window
  <option value="yes">New window
</select>
<br><br>
<select style="background-color: #ff0030; color: #00ff00; font-size : x-small">
  <option value="no" selected>same window
  <option value="yes">New window
</select>
</form>
</body>
</html>


[Comments based on testing with 2002030710 trunk build, classic skin, win2k,
XBL Form Controls enabled, unless otherwise noted. madhur: could you test this
on the full matrix (Classic|Modern) X (win32|MacOS9|MacOSX|Linux) and update
this bug with the status on those platform/skin combinations.  Thanks.]

Comment 1

[John Morrison]

Attachment: testcase; popups should be green text on red background (beauoootiful).

Comment 2

[John Morrison]

adding nsbeta1. I think we do need to consider each of these for machv if we 
are to do xbl-based form controls.

Comment 3

[Brian Ryner (not reading)]

popups should set background-color: inherit

Comment 4

[David Hyatt]

Attachment: Fix the popup to inherit the background color.

Comment 5

[David Hyatt]

--> me

Comment 6

[Brian Ryner (not reading)]

Comment on attachment 73653 [details] [diff] [review]
Fix the popup to inherit the background color.

r=bryner

Comment 7

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 73653 [details] [diff] [review]
Fix the popup to inherit the background color.

sr=[e-mail]

Comment 8

[David Hyatt]

Fixed.

Comment 9

[:deanis74]

The background color on forms is white and in the testcase is red.  So I guess
this is fixed.  Is there another bug open for the text color not being inherited
(see testcase) ?