1021110 - Icons in about:newtab's search panel should be high-res (2x DPI) on high-res displays

Status: VERIFIED FIXED

(Firefox :: New Tab Page, defect)

Description

[Drew Willcoxon :adw]

On my high-res MacBook, the icons in about:newtab's search panel are fuzzy.  They should be 2x DPI or whatever the right term is, like how they are in the standard search bar in the browser's toolbar.  The search bar is drawing from the same data that about:newtab is, so I'm not sure what needs to happen.  Matt mentioned that icons in browser tabs are chosen using -moz-resolution, so maybe that's the way to do it.

Comment 1

[Drew Willcoxon :adw]

Attachment: patch

That was incredibly easy.  #-moz-resolution is magic.  This is based on how the search bar's icons are defined: http://mxr.mozilla.org/mozilla-central/source/browser/components/search/content/search.xml?rev=1061661a1119#296  Tab icons work the same way: http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml?rev=20f9b816ebcd#834

newtab's larger search logos should probably work the same way.

Comment 2

[Drew Willcoxon :adw]

Marco, could we add this to the current iteration, please?

Re-evaluating at p=2 since this hasn't been prioritized yet.  A little larger than a 1 since it took some time to figure out, but smaller than a 3 because the implementation was so simple.

Comment 3

[Marco Mucci [:MarcoM]]

Added to Iteration 33.1

Would you recommend a [qa+] or [qa-] for verification?

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

Comment on attachment 8441692 [details] [diff] [review]
patch

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

Fancy. Did you verify that this works with object URLs?

Comment 5

[Drew Willcoxon :adw]

Thanks for the quick review.  Yes, it looks great, much better than before.

https://hg.mozilla.org/integration/fx-team/rev/ee97689a0506

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ee97689a0506

Comment 7

[Marco Mucci [:MarcoM]]

Hi Florin, can a contact be assigned for QA verification.

Comment 8

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

Juan, could you verify this on your side since we do not have a Retina device here?

Comment 9

[juan becerra [:juanb]]

Verified in latest nightly using a MB with Retina display running 10.9.x. Icons in the about:newtab page look sharp.