Closed Bug 1169683 Opened 9 years ago Closed 9 years ago

The chromeUI icon in the identity-popup is blurry on Windows HiDPI

Categories

(Firefox :: Theme, defect)

defect
Not set
normal
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 41
Iteration:
41.2 - Jun 8
Tracking Status
firefox40 --- fixed
firefox41 --- fixed

People

(Reporter: jaws, Assigned: jaws)

References

Details

Attachments

(2 files, 1 obsolete file)

      No description provided.
Flags: qe-verify-
Flags: firefox-backlog+
Attached patch PatchSplinter Review
Attachment #8612957 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8612957 [details] [diff] [review]
Patch

Review of attachment 8612957 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/themes/shared/identity-block.inc.css
@@ +161,5 @@
>    }
> +
> +  #identity-popup.chromeUI > #identity-popup-container > #identity-popup-icon {
> +    list-style-image: url("chrome://branding/content/icon128.png");
> +    -moz-image-region: rect(0, 128px, 128px, 0);

NB: the -moz-image-region is needed in the shared file for the chromeUI because we don't have HiDPI icons for identity.png on Linux and Windows. It's also defined in the OSX file for #identity-popup-icon but I don't want to move that generic rule here since we are missing the identity@2x.png image and it will cause the 1x graphic to look worse in the interim.
Attachment #8612957 - Flags: review?(gijskruitbosch+bugs) → review+
Attachment #8612984 - Attachment is obsolete: true
Attachment #8612984 - Flags: review?(ehsan)
https://hg.mozilla.org/mozilla-central/rev/59226e937b85
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Attached patch Patch for 40Splinter Review
Approval Request Comment
[Feature/regressing bug #]: Windows 10 HiDPI
[User impact if declined]: HiDPI icons on Windows are blurry
[Describe test coverage new/current, TreeHerder]: on mozilla-central for much of 41-nightly
[Risks and why]: none expected 
[String/UUID change made/needed]: none

https://hg.mozilla.org/try/pushloghtml?changeset=71084a9edf1e
Attachment #8627207 - Flags: approval-mozilla-beta?
Comment on attachment 8627207 [details] [diff] [review]
Patch for 40

Visual fix in support of Windows 10. On m-c for more than a month. Beta+
Attachment #8627207 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.