885065 - (australis) Icons too large, rendering them ugly

Status: RESOLVED WORKSFORME

(Firefox :: Theme, defect)

Description

[Markus Popp]

Attachment: Icons with Australis UI

Icons with Australis UI are too large, making them somewhat blurry, certainly ugly. See screenshots.

Comment 1

[Markus Popp]

Attachment: Same icons, current UI (without Australis)

For comparison, here is what the icons look like now, without Australis (same scale)

Comment 2

[Justin Dolske [:Dolske]]

P2 on the assumption this is Linux-wide.

Comment 3

[Johan C]

Attachment: Large icons on windows.

Some of the toolbarbuttons have large icons on windows.

When placed in the navbar they expand the height of it (see screenshot).

The large toolbarbuttons in the screenshot have the following two attributes (the toolbarbuttons with small icons do not):
"widget-id"
"widget-type"

The following toolbarbuttons have icons that are too large:
"Open file"
"Save this page"
"Open a new Private Browsing window"
"Add-ons manager"
"Toggle Developer Tools"
"Preferences..."
"Find in this page"

Comment 4

[Markus Popp]

Bug 828819 may also be worth considering along with this issue.

Comment 5

[:Gijs (he/him)]

Is this bug (purely about the size) actually still reproducible with the latest version of UX? The icons are still not the ones we want to have on Linux in the final version - still waiting for those - but the sizing problem should be fixed now. Markus, is that correct?

Comment 6

[Markus Popp]

Attachment: Addon icons in July 5 UX builds.

Here is what they look like in today's UX Nightly.

Comment 7

[:Gijs (he/him)]

(In reply to Markus Popp from comment #6)
> Created attachment 771657 [details]
> Addon icons in July 5 UX builds.
> 
> Here is what they look like in today's UX Nightly.

OK. That looks OK to me considering that small icons mode was removed. Trunk looks the same when you switch to large icons mode. The reason the add-on icons are now small is probably that you used the same profile on (normal, trunk) Nightly at some point, switched back to small icons mode (because UX migrates that and turns it off), and the add-on icons still have CSS rules for that, but actual trunk doesn't.

Based on that and the UX built-in icons looking the way they should, marking this WFM.