Style toolbarbutton[type="menu"]

RESOLVED FIXED in Firefox 4.0b8

Status

()

Firefox
Theme
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: mstange, Assigned: mstange)

Tracking

Trunk
Firefox 4.0b8
All
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

8 years ago
Created attachment 459368 [details] [diff] [review]
v1

Give them the right dropmarker image and fix up the text for extensions that display it inside the button, like Feedback/Input.
Attachment #459368 - Flags: review?(dao)
Comment on attachment 459368 [details] [diff] [review]
v1

Please use .toolbarbutton-1 instead of toolbarbutton. We require the toolbarbutton-1 class right now so that items can opt out of the primary toolbarbutton appearance. type="menu-button" is an unfortunate exception right now.

Even if we wanted to change this, we'd want to do it consistently across platforms.

>+toolbar[mode="icons"] toolbarbutton[type="menu"]:not(.bookmark-item) > .toolbarbutton-text {
>+  font: -moz-dialog;
>+  margin-top: 0;
>+  -moz-margin-start: 3px;
>+  -moz-margin-end: 1px;
>+}

The Feedback button is only relevant for betas and doesn't seem to follow a common pattern, so I don't really want this code here. If we are serious about supporting text in toolbarbuttons in icons mode, we should probably provide a class for this.
Attachment #459368 - Flags: review?(dao) → review-
(Assignee)

Comment 2

8 years ago
Created attachment 477313 [details] [diff] [review]
v2
Attachment #459368 - Attachment is obsolete: true
Attachment #477313 - Flags: review?(dao)
Comment on attachment 477313 [details] [diff] [review]
v2

>+.toolbarbutton-1 > .toolbarbutton-menu-dropmarker,
> .toolbarbutton-1 > .toolbarbutton-menubutton-dropmarker {
>   list-style-image: url(chrome://browser/skin/toolbarbutton-dropmarker.png);
>+}
>+
>+.toolbarbutton-1  > .toolbarbutton-menu-dropmarker {
>+  -moz-margin-end: 1px;
>+}
>+
>+toolbarbutton[type="menu-button"] > .toolbarbutton-menubutton-dropmarker {
>   width: 14px;
>   padding-top: 2px;
>   -moz-border-start: none !important;
> }

Seems like toolbarbutton[type="menu-button"] should be .toolbarbutton-1.
(Assignee)

Comment 4

8 years ago
Created attachment 477493 [details] [diff] [review]
v3

Sometimes I feel blind.
Attachment #477313 - Attachment is obsolete: true
Attachment #477493 - Flags: review?(dao)
Attachment #477313 - Flags: review?(dao)
Comment on attachment 477493 [details] [diff] [review]
v3

>+.toolbarbutton-1  > .toolbarbutton-menu-dropmarker {

redundant space after .toolbarbutton-1
Attachment #477493 - Flags: review?(dao) → review+
(Assignee)

Updated

8 years ago
Attachment #477493 - Flags: approval2.0?

Updated

7 years ago
Attachment #477493 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 6

7 years ago
http://hg.mozilla.org/mozilla-central/rev/207ba2088b65
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
You need to log in before you can comment on or make changes to this bug.