Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set font size for UI #50

Open
leshek-pawlak opened this issue Dec 9, 2016 · 7 comments
Open

Set font size for UI #50

leshek-pawlak opened this issue Dec 9, 2016 · 7 comments

Comments

@leshek-pawlak
Copy link

Hi!
First of all - this theme is great! I really love the design, transitions and colors.

But.. there is one thing which makes it quite cumbersome to use for me - font size.
It's set to 14px. It's too big for me.
To change the font size in the entire theme, I need to add some nasty css code to my atom stylesheet.

Is it possible to add font-size selection in the theme settings?
This option is for example in the "one-dark-ui" theme.

Greetings!

@arcticicestudio
Copy link
Contributor

Hi, nice to hear you like it 😄
Your request was also on my TODO list, only too busy with my job to create a issue for it yet, but I will definitely take a look at it this weekend as it shouldn't be too much effort to implement.

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Dec 10, 2016

I implemented the new theme setting "UI Font Size".
You can check out the feature/ghi-#50-theme-setting-font-size branch to test it and see if it fits your needs.
I limited the range of the possible value that can be set from 8 to 20 to keep the theme clean and readable and to avoid accidentally oversized font which can only be changed by manually editing the config.cson file.

The current implementation does exactly what the name of the settings describes: I only changes the font size of the UI.
Some other themes like "One Dark UI" are also scaling all UI components based on the font size value which is in my eyes not the way this setting should work.
If you'd like to scale the UI or having problems with HIDPI the --force-device-scale-factor=<FLOATING_POINT_VALUE> (e.g.: --force-device-scale-factor=1.5) option of Atom is the correct way to go.

As far as I tested it works fine except the fact that there is a alignment issue of the theme selection drop-downs.

Feel free to post your review and/or improvement suggestions.

Screenshots
(The blurry font in the textbox is related to my local HIDPI settings)

Default 14px

Custom 10px

@arcticicestudio
Copy link
Contributor

It seems like there are edge cases where the default @font-size LESSCSS variable is not used for some of the UI elements like the "Open Config Folder" on the screenshots.
As of the migration to the new Atom theme package API in versions >=1.13.0 I also recognized deformed UI components which may be related to the "hardcoded" px values 😐

Contrary to my statement above I'll maybe have to revisit this feature and refactor all selectors and rules that are related to the font size and implement it the way the official One Dark UI theme is designed by using multiplier to also scale/resize elements to avoid unexpected size behaviours.

@bencardoen
Copy link

Thanks for implementing this, works great for me.

@we3geeks
Copy link

Would love to see this change merged and released. (Or perhaps I'm just not seeing it for some reason?)

@russoue
Copy link

russoue commented Aug 24, 2017

Is there any ETA to merge and release it?

@gamblebkb
Copy link

These seem related so I'll post it here.
Changing font family in stylesheet(styles.less) doesn't work for this theme.
I've tried also with default ones and it works.
Thanks!

@arcticicestudio arcticicestudio removed their assignment Dec 6, 2018
@arcticicestudio arcticicestudio added the Hacktoberfest This repository participates in the Hacktoberfest label Oct 8, 2020
@arcticicestudio arcticicestudio removed the Hacktoberfest This repository participates in the Hacktoberfest label Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants