Closed Bug 672050 Opened 13 years ago Closed 13 years ago

Draw toolbar buttons with CoreUI

Categories

(Core :: Widget: Cocoa, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: mstange, Assigned: mstange)

References

Details

Attachments

(2 files, 1 obsolete file)

Attached patch wip (obsolete) — 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.
Ah, then I guess we don't need any patch in bug 553992?
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.
OK. Awesome work Markus :-)
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.)
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)
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)
Attachment #549334 - Flags: review?(dao) → review+
Depends on: 676387
Attachment #549331 - Flags: review?(joshmoz) → review+
http://hg.mozilla.org/mozilla-central/rev/4fb70b8c389f

The other patch was backed out from inbound.
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/4c222348890a
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Depends on: 678730
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: