560288 - consolidate hud-button style rules

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

They're duplicated a couple of times in browser.css unnecessarily.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Attachment: patch

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 440014 [details] [diff] [review]
patch

>diff --git a/browser/themes/pinstripe/browser/browser.css b/browser/themes/pinstripe/browser/browser.css

>+.hud-button {

>+  -moz-border-right-colors: rgba(53,53,53,1) rgba(53,53,53,1) rgba(162,162,162,1);

> .editBookmarkPanelHeaderButton,
> .editBookmarkPanelBottomButton {

>-  -moz-border-top-colors: rgba(0,0,0,0.35) rgba(26,26,26,0.5) rgba(255,255,255,0.4);
>-  -moz-border-right-colors: rgba(0,0,0,0.35) rgba(26,26,26,0.5) rgba(255,255,255,0.4);

This -moz-border-right-colors was different than all the others, but I think that was a copy/pasto (from -moz-border-top-colors) rather than an intentional difference.

> #editBMPanel_folderMenuList {

>-  color: #ffffff !important;

The !important is lost here, but it doesn't appear to be needed.

> #editBookmarkPanel .expander-up,
> #editBookmarkPanel .expander-down {

>   -moz-border-radius: 5px;

This is left to override the hud-button default (20px).

>-  color: #ffffff !important;

!important also lost here, but these buttons have no text so I think that doesn't matter.

Comment 3

[:Gavin Sharp [email: [e-mail]]]

(In reply to comment #2)
> This -moz-border-right-colors was different than all the others, but I think
> that was a copy/pasto (from -moz-border-top-colors) rather than an intentional
> difference.

Also I wasn't able to see a visual difference caused by the discrepancy, so I wonder whether these rules have any effect at all. Or maybe I was just looking at the wrong buttons.

Comment 4

[Dão Gottwald [:dao]]

I don't think I like hud-button as a class. It seems to be set only based on whether pinstripe styles a panel black, but other themes might make different choices.

We could use http://mxr.mozilla.org/mozilla-central/source/browser/themes/pinstripe/browser/shared.inc to consolidate the styling without a class.

Comment 5

[Dão Gottwald [:dao]]

Comment on attachment 440014 [details] [diff] [review]
patch

>diff --git a/browser/components/places/content/editBookmarkOverlay.js b/browser/components/places/content/editBookmarkOverlay.js

>   toggleTagsSelector: function EIO_toggleTagsSelector() {
>     var tagsSelector = this._element("tagsSelector");
>     var tagsSelectorRow = this._element("tagsSelectorRow");
>     var expander = this._element("tagsSelectorExpander");
>     if (tagsSelectorRow.collapsed) {
>-      expander.className = "expander-up";
>+      expander.className = "expander-up hud-button";

This sets hud-button in the Library too, which certainly seems wrong.

Comment 6

[:Gavin Sharp [email: [e-mail]]]

Attachment: %define patch

Like this then? I don't think preprocessor.py supports multi-line #defines.

Comment 7

[Dão Gottwald [:dao]]

Looks ok to me. I'd capitalize "Button" and include the semicolon in the %define.

Comment 8

[:Gavin Sharp [email: [e-mail]]]

Attachment: %define patch, v2

Comment 9

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/6b4fa780bc5c