Open Bug 1247390 Opened 8 years ago Updated 8 months ago

Use the correct notification favicon resolution on OS X

Categories

(Toolkit :: Alerts Service, defect, P3)

All
macOS
defect

Tracking

()

People

(Reporter: lina, Unassigned)

References

(Blocks 2 open bugs)

Details

See bug 1224785, comment 13.

For a multi-resolution favicon (attachment 8463655 [details]), "32" shows up for both 1x and 2x resolution. I can call `[icon setSize:NSMakeSize(16, 16)]`, but that'll force "16" on a 2x display.

https://dxr.mozilla.org/mozilla-central/source/widget/cocoa/nsMenuItemIconX.mm loads the image, then decodes the first frame and calls `CreateNSImageFromCGImage`. That doesn't seem to help, though; if I load the icon as a `moz-anno:favicon:` URL, it still shows "32" on a 1x display.
:seth, flagging you for assistance, when you have some spare cycles. Thanks!
Flags: needinfo?(seth)
Blocks: 1249808
Flags: needinfo?(seth.bugzilla)
OS: Unspecified → macOS
Priority: -- → P3
Hardware: Unspecified → All
Severity: normal → S3

This might make more sense under Core :: Widget: Cocoa, so feel free to move there too.

Component: Notifications and Alerts → Alerts Service
You need to log in before you can comment on or make changes to this bug.