Closed Bug 1093312 Opened 6 years ago Closed 6 years ago

Disclosure button focus ring could be a bit better

Categories

(Core :: Widget: Cocoa, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: stefanh, Assigned: stefanh)

Details

Attachments

(1 file)

Attached patch Fix focus ringSplinter Review
Unless you look very carefully you won't notice it, but the focus ring looks a little bit crammed. That's because I totally forgot about nsNativeThemeCocoa::GetWidgetOverflow in bug 1085134 :-(
Attachment #8516256 - Flags: review?(mstange)
Component: Widget → Widget: Cocoa
Attachment #8516256 - Flags: review?(mstange) → review+
https://hg.mozilla.org/mozilla-central/rev/afb437f1ee02
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.