Drew Willcoxon :adw Assignee
	Description • 10 years ago

All the search providers shown in about:newtab by default have both low- and high-DPI logo images except for Yahoo, which has only a high-DPI image (landed in bug 1009299).  The reason is that I wanted to let bug 1009266 downsample the image instead, but there are a couple of problems with that: (1) the high-DPI image is on 31 but bug 1009266 is on 32, so on 31, the Yahoo image just doesn't appear at all, and (2) bug 1027791 points out that downsampling is a bad UX right now and may even hurt performance.

	Drew Willcoxon :adw Assignee
	Comment 1 • 10 years ago

For the record, in addition to bug 1027791, Avi talks about the Yahoo logo downsampling problems in specific in bug 1019990 comment 16 and earlier.

	Drew Willcoxon :adw Assignee
	Comment 2 • 10 years ago

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

	Matthew N. [:MattN]
	Comment 3 • 10 years ago

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

Thanks. I was going to ask about the inconsistency in the past.

	Marco Mucci [:MarcoM]
	Comment 4 • 10 years ago

Added to Iteration 33.2

	Drew Willcoxon :adw Assignee
	Comment 5 • 10 years ago

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

	Sylvestre Ledru [:Sylvestre]
	Comment 6 • 10 years ago

Tracking because it is a new feature and adding an icon should be low risk.

	Carsten Book [:Tomcat]
	Comment 7 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/725e50a3bfc1

	Drew Willcoxon :adw Assignee
	Comment 8 • 10 years ago

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

[Approval Request Comment]
Bug caused by (feature/regressing bug #):
Bug 962490 added a search field to about:newtab and landed on 31.  The logo of the currently selected search engine, if available, appears to the left of the search field.

User impact if declined:
On 31, when Yahoo is the currently selected engine, its logo will not appear if the user's display is low-DPI.  about:newtab gracefully handles missing logos, though.  On 32, the logo will appear, but the user may notice a slight flicker due to downsampling the high-DPI logo as described in bug 1009299.

Testing completed (on m-c, etc.):
manual testing, automated testing on m-c

Risk to taking this patch (and alternatives if risky):
very low

String or IDL/UUID changes made by this patch:
none

	Drew Willcoxon :adw Assignee
	Comment 9 • 10 years ago

Bug 998071 landed on 32 and changed yahoo.xml, which made the m-c/Aurora patch not apply cleanly to Beta.

[Approval Request Comment]
see comment 8

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/8eff2a44f2b2
https://hg.mozilla.org/releases/mozilla-beta/rev/0c0706e210c3

	Petruta Horea [:phorea], Desktop QA
	Comment 11 • 10 years ago

Verified that the Yahoo logo is displayed on about:newtab page when it is chosen from the search list.

Environments: Win 7 64-bit, Ubuntu 12.10 32-bit, Mac OSX 10.9.3
Builds: Firefox 31 beta 5, latest Aurora 20140629081512, latest Nightly 20140629030206