Closed Bug 456595 Opened 16 years ago Closed 16 years ago

Plain readonly textboxes should have a transparent background

Categories

(Toolkit :: Themes, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED

People

(Reporter: florian, Assigned: florian)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Simple patch (obsolete) — Splinter Review
Read-only textboxes are often used to allow the user to select the content, and copy/paste/use the context menu.

The plain style class of textboxes is used to display a textbox with no border or margin.

When a readonly textbox has the class plain, on Mac the background color (-moz-Dialog, set by the style rule for textbox[readonly="true"]) of the textbox is visible. On Linux and Windows, this is usually not the case because -moz-Dialog tends to match the background color of most windows.

I think this background should be transparent.
Attachment #339983 - Flags: review?(gavin.sharp)
Where (i.e. which product/UI) has this come up as an issue?
Actually, with non default OS themes, I can see this on Windows and Linux too.

(In reply to comment #1)
> Where (i.e. which product/UI) has this come up as an issue?

Currently I have it in 2 different places in Instantbird but I originally saw this in Firefox.

A few examples in Firefox:
 - The bottom of the View Cookies dialog
 - The textbox at the bottom of the Application Details dialog (from the Applications tab of the Preference dialog)

I think it's also visible in DOM Inspector but I haven't checked right now.

I was pretty sure it used to be an issue in Page Info too but I just checked and saw that the code doesn't use the class plain and duplicates the '-moz-appareance: none' style rule instead.
Assignee: nobody → florian
Attachment #339983 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #340803 - Flags: review?(gavin.sharp)
Attachment #339983 - Flags: review?(gavin.sharp)
OS: Mac OS X → All
Hardware: Macintosh → All
Summary: Plain readonly textboxes look ugly on mac. → Plain readonly textboxes should have a transparent background
I think one of the Neils would probably be a better reviewer for this.
Attachment #340803 - Flags: review?(gavin.sharp) → review?(enndeakin)
Attachment #340803 - Flags: review?(enndeakin) → review+
Pushed: http://hg.mozilla.org/mozilla-central/rev/a269caae011a
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: