Cornel Ionce [:noni] [Hubs QA] Reporter
	Description • 9 years ago

Reproduced on a Microsoft Surface Pro 2 device running Windows 10 64-bit.
It reproduces with:
* latest Nightly, 
* latest DevEdition, build ID: 20150729004002.

STR:
1. Open Firefox
2. Set the DevEdition dark theme (about:addons -> Appearance -> enable Developer Edition).
3. Hover over the dropdown icon located in the location bar.

Results:
The dropdown icon is very small on hover.

Screenshot: http://i.imgur.com/1jVHkOy.png

Notes:
* This issue only reproduces on HiDPI devices.
* Only the DevEdition dark theme is affected.
* This issue is a regression.

Regression Range:
Last-good buildID: 20150528030206
First-bad buildID: 20150529030205

Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=baa9c64fea6f&tochange=f986e55c4e0b

	Dão Gottwald [:dao]
	Comment 1 • 9 years ago

Not tracking DevEdition issues as part of bug 1077146.

	Dão Gottwald [:dao]
	Comment 2 • 9 years ago

Probably caused by bug 1023607.

	Tim Nguyen :ntim
	Comment 3 • 9 years ago

I feel like we should just use the devedition SVG for the Windows theme. The color can be easily adjusted per platform, and the shape was based on the Windows' PNG

	Tim Nguyen :ntim
	Comment 4 • 9 years ago

(In reply to Tim Nguyen [:ntim] from comment #3)
> I feel like we should just use the devedition SVG for the Windows theme. The
> color can be easily adjusted per platform, and the shape was based on the
> Windows' PNG

Meant Windows version, not platform

	Brian Grinstead [:bgrins]
	Comment 6 • 9 years ago

Comment on attachment 8641860 [details] [diff] [review]
Patch

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

This fixes the problem, thanks!

	Pulsebot
	Comment 7 • 9 years ago

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

	Carsten Book [:Tomcat]
	Comment 8 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/18785164404c

	Cornel Ionce [:noni] [Hubs QA] Reporter
	Comment 9 • 9 years ago

Confirming the fix on latest Nightly, build ID: 20150804030204 using a MS PRO 2 device.

	:Gijs (he/him)
	Comment 10 • 9 years ago

Can/should we uplift to 41? Might not be worth it considering merge day is Monday and we don't care about devedition on beta, I guess...?

	Jared Wein [:jaws] (please needinfo? me) Assignee
	Comment 11 • 9 years ago

Comment on attachment 8641860 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/regressing bug #]: bug 1023607
[User impact if declined]: UI glitch in the URL bar
[Describe test coverage new/current, TreeHerder]: manual testing
[Risks and why]: no risk, simple CSS fix
[String/UUID change made/needed]: none

There will still be a couple builds of DevEdition41 before the merge and this is a really simple fix so we should uplift it if possible.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 12 • 9 years ago

Comment on attachment 8641860 [details] [diff] [review]
Patch

Verified on Nightly. Aurora+

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 9 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/ba9b666f6429

	Cornel Ionce [:noni] [Hubs QA] Reporter
	Comment 14 • 9 years ago

I've also verified this issue on latest DevEdition, build ID: 20150811004008.