Consider using ThreeDShadow instead of ThreeDDarkShadow for the border for the location bar and search box in Windows Classic

VERIFIED FIXED in Firefox 6

Status

()

Firefox
Theme
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Kai Liu, Assigned: Kai Liu)

Tracking

Trunk
Firefox 6
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

6 years ago
Created attachment 532545 [details]
Before and after

In bug 560755, the border for the location bar and search box was changed to ThreeDDarkShadow because the previous setting, a partially transparent black border, did not show up on a black background.

The downside of using ThreeDDarkShadow is that in most classic color schemes, it produces a completely black border, which is very jarring and inconsistent with the rest of the style of that upper region of the browser: no other border is nearly that dark, and it really stands out like a sore thumbs (especially when it is juxtaposed with the borders of the adjacent toolbar buttons).

As a compromise, I think we should use ThreeDShadow instead.  Its color more closely matches that of the original partially transparent black, and it is still visible in a black high-contrast scheme.  It doesn't stand out as much in HC, but none of the other borders do either (e.g., look at the border of tabs in black high-contrast), and most importantly, it still remains visible.

Attached is a screenshot showing the before-and-after of this proposed change:

* Normal classic: Note how incongruous the black border looks when juxtaposed with the nearby toolbar button borders and tab borders.

* High-contrast classic: The border is still visible, and although it does not stand out as much as before, none of the other borders in that region of the interface--tabs, buttons, etc.--do either.

* Themed: Just a sanity check to show that this is indeed a classic-only change.
(Assignee)

Comment 1

6 years ago
Created attachment 532546 [details]
example showing that a gray-on-black text box is not unprecedented

This is just to show that there are already other prominent bits of text-input UI that use a gray border in a black high-contrast scheme...
(Assignee)

Comment 2

6 years ago
Created attachment 532547 [details] [diff] [review]
trivial patch
Attachment #532547 - Flags: review?(dao)

Updated

6 years ago
Attachment #532547 - Flags: review?(dao) → review+
(In reply to comment #0)
> It doesn't stand out as much
> in HC, but none of the other borders do either (e.g., look at the border of
> tabs in black high-contrast)

(In reply to comment #1)
> Created attachment 532546 [details]
> example showing that a gray-on-black text box is not unprecedented
> 
> This is just to show that there are already other prominent bits of
> text-input UI that use a gray border in a black high-contrast scheme...

The theme doesn't really adjust to the OS theme at all there. Obviously not great examples -- we want to do better than that if we can.
http://hg.mozilla.org/mozilla-central/rev/f388d1d334ab
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 6
Mozilla/5.0 (Windows NT 5.1; rv:6.0a1) Gecko/20110519 Firefox/6.0a1

Verified the change with the help of before/after image.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.