Closed
Bug 672050
Opened 13 years ago
Closed 13 years ago
Draw toolbar buttons with CoreUI
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: mstange, Assigned: mstange)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
jaas
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
If we want the right toolbar button appearance on 10.7, we can either add special cases to our CSS and use images to get the noise texture, or we can ask the system to draw the buttons.
This patch (on top of the one in bug 668195) implements -moz-appearance:toolbarbutton with CoreUI and contains almost all of the necessary CSS changes.
The only thing I haven't fixed up yet is the circle back button.
Comment 1•13 years ago
|
||
Ah, then I guess we don't need any patch in bug 553992?
Assignee | ||
Comment 2•13 years ago
|
||
I think we should take your patch in bug 553992 anyway since I don't know when this bug will be ready. We can always back it out again when I'm finished here.
This bug is blocked on bug 668195, and that needs some more theme fixup before it can land, and I don't know when I'll have time for that.
Comment 3•13 years ago
|
||
OK. Awesome work Markus :-)
Comment 4•13 years ago
|
||
OT: Markus, do you think CoreUI might be a viable option for drawing
scrollbars (instead of the HITheme API we're currently using)? And
might this allow us to support autohiding the scrollbars on OS X 10.7
(which the HITheme API currently doesn't support)?
(This is in regard to bug 636564.)
Assignee | ||
Comment 5•13 years ago
|
||
Tabs and segmented toolbar buttons can be rendered with CoreUI with almost the same code, using widget types "tab" and "kCUIWidgetButtonSegmentedSCurve", respectively.
Assignee: nobody → mstange
Attachment #546325 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #549331 -
Flags: review?(joshmoz)
Assignee | ||
Comment 6•13 years ago
|
||
The CSS patch that will convert our custom toolbarbutton CSS to -moz-appearance: toolbarbutton will also set -moz-appearance: toolbarbutton on the dropmarker of a toolbarbutton[type="menu-button"]. Open menu button toolbarbuttons should have a pressed dropmarker, and inheriting open into the dropmarker is simpler than making widget code check the dropmarker's parent.
Attachment #549334 -
Flags: review?(dao)
Updated•13 years ago
|
Attachment #549334 -
Flags: review?(dao) → review+
Attachment #549331 -
Flags: review?(joshmoz) → review+
Assignee | ||
Comment 7•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/0a6140e88a2d
http://hg.mozilla.org/integration/mozilla-inbound/rev/4fb70b8c389f
Whiteboard: [inbound]
Comment 8•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/4fb70b8c389f
The other patch was backed out from inbound.
Whiteboard: [inbound]
Comment 9•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in
before you can comment on or make changes to this bug.
Description
•