Updating to 8.x-1.2 caused the font size on my site to increase on admin pages.

CommentFileSizeAuthor
#9 Screen Shot 2016-11-10 at 16.50.58.png61.3 KBidiaz.roncero
#9 Screen Shot 2016-11-10 at 16.51.12.png72.5 KBidiaz.roncero
#8 2820992-adminial-theme-remove-zoom.patch532 bytesBcwald
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

danthorne created an issue. See original summary.

ANDiTKO’s picture

ANDiTKO CreditAttribution: ANDiTKO as a volunteer commented

Yes that was intentional. Don't you like it?
I'm planning to make this to be optional in next update, but I'm not sure which setting to be default.
I think "seven" fonts are pretty small.

danthorne’s picture

danthorne
Primary language English
Location Devon, UK
CreditAttribution: danthorne commented

Not when the font size for the toolbar is smaller on the frontend.

I definitely think the option needs to be added.

But I actually don't dis-mind the larger font size :)

kopin’s picture

kopin CreditAttribution: kopin commented

I agree, it looks like I'm using enlarged mode for blind people.

The zoom 1.1 style also brakes some Editor functionality. Now i'm overriding it trough a subtheme

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
CreditAttribution: saltednut at Acquia commented

Don't you like it?

We don't like it.

Is there a way this can be turned off by default?

Worth noting that we love the theme overall. :)

Bcwald’s picture

Bcwald CreditAttribution: Bcwald commented

Why would you do this through zoom? That is a really intrusive method and breaks many modules that provide their own CSS that are meant to be contained to itself. If you want to increase the font, you should do that via body font style.

modulist’s picture

modulist CreditAttribution: modulist commented

I would recommend increasing the size of the headers for greater contrast.

However, keeping the size of the font and the vertical padding down in tables will make the rows more compact, allowing you to view more rows in a typical browser window.

Bcwald’s picture

Bcwald CreditAttribution: Bcwald commented
FileSize
2820992-adminial-theme-remove-zoom.patch532 bytes
idiaz.roncero’s picture

idiaz.roncero
Primary language Spanish
Location Madrid
CreditAttribution: idiaz.roncero commented
FileSize
Screen Shot 2016-11-10 at 16.51.12.png72.5 KB
Screen Shot 2016-11-10 at 16.50.58.png61.3 KB

I agree this has to be an optional. Although I like the idea, I also felt the approach is prone to bugs.

Using the zoom property is easier than changing all the font sizes and spacings, but it leads to some unexpected behaviors like this, in the CKeditor:

Without zoom:1.1

Original

With zoom:1.1

With zoom

And in general, UI images (like the throbber and the ckeditor buttons) feel blurred because of the zoom.

ANDiTKO’s picture

ANDiTKO CreditAttribution: ANDiTKO at Netstudio commented
Status: Active » Patch (to be ported)

The zoom will be reverted back as default in next release. Stay tuned.
Feel free to use the patch before its ported. Thank you for your input

lukedekker’s picture

lukedekker CreditAttribution: lukedekker commented

The zoom also causes the height of some Views admin overlays to be miscalculated. This causes parts of the form to display off window.

I know this has already been resolved, just an FYI for consideration on future updates.

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
CreditAttribution: saltednut at Acquia commented

Thanks for all your hard work @ANDiTKO !

ANDiTKO’s picture

ANDiTKO CreditAttribution: ANDiTKO at Netstudio commented
Status: Patch (to be ported) » Fixed

Zoom is now removed. But i like the idea of controlling font-size and make things a little bigger. Next time will be a setting and i will find a way to do it without using the zoom property.

Thanks everyone for testing and reporting!

ANDiTKO’s picture

ANDiTKO CreditAttribution: ANDiTKO at Netstudio commented
Status: Fixed » Closed (fixed)
idiaz.roncero’s picture

idiaz.roncero
Primary language Spanish
Location Madrid
CreditAttribution: idiaz.roncero commented

I do like the idea also. Maybe you could try setting font-size as a percentage (i.e. 110%)?