Closed Bug 460681 Opened 16 years ago Closed 16 years ago

Disabled link toolbar buttons need more three-dimensional appearance

Categories

(SeaMonkey :: Themes, defect)

x86
Windows 2000
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

References

Details

(Keywords: polish)

Attachments

(1 file)

Bug 69710 introduced a three-dimensional disabled appearance for toolbarbuttons when using the Windows Classic skin. However, the rules for the bookmarks and link toolbar buttons conflict with this, resulting in disabled link toolbar buttons being drawn using incorrect system colours.
Attached patch Proposed patchSplinter Review
This patch resolves the problem by not changing the appropriate CSS styles on disabled bookmarks thus avoiding having to override them with the wrong values.

As the underlying CSS is OS-specific I'd like cross-platform testing :-)
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #343833 - Flags: review?(iann_bugzilla)
Comment on attachment 343833 [details] [diff] [review]
Proposed patch

See previous comment, because I don't know your unique username, assuming you have one...
Attachment #343833 - Flags: review?(stefanh)
Comment on attachment 343833 [details] [diff] [review]
Proposed patch

This seems to work fine on mac. That is, it doesn't change anything.
Attachment #343833 - Flags: review?(stefanh) → review+
Comment on attachment 343833 [details] [diff] [review]
Proposed patch

Works fine on Linux, i.e. no change, and the change seems fairly subtle on windows, hard to tell if it is my eyes or there is a change.
Attachment #343833 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 90692f79a317 to comm-central.
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: