Go back button doesn't scale properly on some systems about firefox-gnome-theme HOT 5 CLOSED

The offending rule is:

#back-button > .toolbarbutton-icon {
	width: 30px !important;
}

The standard rule from browser.css is:

#back-button > .toolbarbutton-icon {
	width: calc(2 * var(--toolbarbutton-inner-padding) + 16px);
}

This results in a computed value of 26px (16px, plus 5+5px padding) instead of 30px.

I think you'll find that the "some systems" seeing this problem have "Compact" density selected in Firefox's "Customise" UI instead of "Normal" - removing the above rule fixes the oversized/blurry icon in that case ;)

On the other hand, with "Normal" density the back icon looks undersized without the 30px rule.

HTH

from firefox-gnome-theme.

Hey, you're totally right. With this theme, the density is no longer important, so changing it back to 'Normal' permanently fixes the issue.

from firefox-gnome-theme.

I can confirm that, i noticed the same behavior.

from firefox-gnome-theme.

I just checked, the back arrow is pixel-identical with either normal density + rule or compact density + no rule.

Maybe make the rule conditional on the current density, if there's a CSS attribute/class for it?

from firefox-gnome-theme.

Well then I'd say just switching to normal density is a good solution. Compact and touch density aren't supported anyway, they break paddings in more places than just this icon and when you switch to touch density everything looks bad. All the icons are underscaled, all buttons are fat, tab close buttons shrink to like 1px high.

I would rather focus on expanding support to Firefox 58, which is gonna be released on January 28, than debugging everything to fix those paddings which can be disabled in 2 clicks by the user and are disabled by default anyway. Also I noticed the icon is still too small even using normal density on Firefox 58+, so that's something which must be fixed before the FF58 release. Also on FF58 the headerbar isn't high enough (probably the reason why the styles for Fedora's CSD patch don't exactly work with FF59) and disabled buttons look too flat. I'll try to fix all these in January. As for Firefox 57, the problem doesn't exist if you use normal density, so I'd say the issue is resolved.

Thank you all for figuring this out and testing the solution, I totally forgot about that density switch.

from firefox-gnome-theme.