Closed Bug 821213 Opened 12 years ago Closed 10 years ago

focus rings too thin in HiDPI mode

Categories

(Core :: Widget: Cocoa, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: heycam, Assigned: mstange)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In HiDPI mode, the blue focus rings shown for <textarea>, <input type=button>, etc. in content seem to be not as thick as in other applications and also not as thick as the corresponding controls e.g. in the address bar and other areas of the window chrome.  (I hesitate to raise this as a bug because I prefer the more subtle blue highlighting!)
Blocks: osx-hidpi
Blocks: 816862
I agree - both that there's a discrepancy, and that I quite like the more subtle version we (inadvertently) have! In the interests of consistency with the platform, though, we should try to fix this.

There are hints in the webkit/chromium code and bug tracker that this is related to CGContext{Get,Set}BaseCTM; see GraphicsContext::platformApplyDeviceScaleFactor in platform/graphics/cg/GraphicsContextCG.cpp.
Attached patch v1Splinter Review
On 10.10 the thinness is much more apparent, so we should probably fix this, even if it looks nice.
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Attachment #8490179 - Flags: review?(smichaud)
By the way, thanks for the pointer to CGContextSetBaseCTM, Jonathan. It worked perfectly.
Comment on attachment 8490179 [details] [diff] [review]
v1

Looks fine to me.
Attachment #8490179 - Flags: review?(smichaud) → review+
https://hg.mozilla.org/mozilla-central/rev/293162286fd2
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Depends on: 1073570
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: