Set an attribute on monochrome toolbars to make life easier for extensions

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
8 years ago
7 years ago

People

(Reporter: Paolo, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

8 years ago
Created attachment 509571 [details] [diff] [review]
The patch

Patch on top of bug 626382. Example add-on stylesheet with this patch:

toolbar[monochrome] #my-button {
  list-style-image: url(icon16-monochrome.png);
}
Attachment #509571 - Flags: review?(dao)
(Reporter)

Comment 1

8 years ago
Created attachment 509609 [details] [diff] [review]
The patch

Revised patch on top of bug 626382.
Attachment #509571 - Attachment is obsolete: true
Attachment #509609 - Flags: review?(dao)
Attachment #509571 - Flags: review?(dao)
(Reporter)

Comment 2

8 years ago
Created attachment 509613 [details] [diff] [review]
The patch

Revised patch on top of bug 626382, the correct file this time.

Sorry for the noise.
Attachment #509609 - Attachment is obsolete: true
Attachment #509613 - Flags: review?(dao)
Attachment #509609 - Flags: review?(dao)

Updated

8 years ago
blocking2.0: betaN+ → ---
Whiteboard: [softblocker]
(Reporter)

Comment 3

8 years ago
Created attachment 509636 [details] [diff] [review]
The patch

Revised patch on top of bug 626382.
Attachment #509613 - Attachment is obsolete: true
Attachment #509636 - Flags: review?(dao)
Attachment #509613 - Flags: review?(dao)

Updated

8 years ago
No longer blocks: 467520
The more I think about this, the more I'm convinced this belongs to theme's chrome.manifest (replacing the counter-reset hack).
I'm not convinced that this bug is the right direction at all. The idea that all icons should be monochrome seems a bit odd, and it's extra load for extension authors.

Comment 6

8 years ago
Wasn't the point that the UX team wanted people to basically make a monochrome version of their theme to 'fit in' with the MacOSX look?

I still haven't heard, by the way, whether there's a plan to deal with there being duplicate themes with different icon sizes (which are bound to turn up), if you remove the option for users to choose between small and large icons.
(Reporter)

Comment 7

8 years ago
(In reply to comment #6)
> Wasn't the point that the UX team wanted people to basically make a monochrome
> version of their theme to 'fit in' with the MacOSX look?

That was my understanding too, and it's mentioned in the article in the bug URL:

http://blog.mozilla.com/addons/2010/12/02/toolbar-buttons-in-firefox-4/

I'm setting the "uiwanted" keyword to get feedback if this is still desired.
Keywords: uiwanted
Comment on attachment 509636 [details] [diff] [review]
The patch

I don't think we should set the expectation that add-on authors need to create various icon versions. This is also not going to work for jetpack-style add-ons. Classic add-ons that want this can target OS X in chrome.manifest.
Attachment #509636 - Flags: review?(dao) → review-

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Keywords: uiwanted
Resolution: --- → WONTFIX
(Reporter)

Comment 9

7 years ago
(In reply to Dão Gottwald [:dao] from comment #8)

Thank you for reviewing this! Since we haven't heard from the UX team on the issue
in a long time, I suppose this is not something they're considering anymore.
You need to log in before you can comment on or make changes to this bug.