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

Scaling down icons globally? #278

Closed
AloisJanicek opened this issue Dec 1, 2017 · 4 comments
Closed

Scaling down icons globally? #278

AloisJanicek opened this issue Dec 1, 2017 · 4 comments
Labels
is:feature Adds or requests new features, or extends existing ones module:ui/modeline Pertains to Doom's :ui modeline module re:fonts Pertains to changing, using and loading fonts status:resolved Issue was addressed internally
Milestone

Comments

@AloisJanicek
Copy link
Contributor

Hi, I am starting out with this awesome Emacs config (and with Emacs in general) and I would like to have icons little bit smaller. I know about all-the-icons-scale-factor, but when I tweak it in my private init.el, it only affects for example neotree and not modeline, because there are explicitly specified local values of all-the-icons-scale-factor in doom-modeline module's config.el.
Do you think it is worth it to introduce additional complexity in form of global variables which would hold scale value for buffer-info and vcs parts of modeline and which user could tweak somewhere in his private module, or should I rather fork doom-modeline's module and tweak it there?
Or maybe there is another way to solve this I am not aware of?

@hlissner
Copy link
Member

hlissner commented Dec 5, 2017

Sadly, there is no way to do this with those local bindings there. I'll look through ui/doom-modeline and see if those are necessary anymore, then you'll be able to use all-the-icons-scale-factor again. I'd prefer to do this than add more global variables.

@hlissner hlissner added this to the 2.1 milestone Dec 5, 2017
@hlissner hlissner added the is:feature Adds or requests new features, or extends existing ones label Dec 5, 2017
@hlissner
Copy link
Member

Sorry for the late response; I've removed the all-the-icon-scale-factor let-bindings in develop, so you should be able to modify that yourself from here on out. As it turns out, they aren't necessary anymore; either the font glyphs have been adjusted since, or I simply can't find the edge cases where they were needed.

Please give it a try. I will keep this open until it has been merged into master.

@hlissner hlissner added the status:resolved Issue was addressed internally label Dec 16, 2017
@hlissner hlissner modified the milestones: 2.1, 2.0.8 Dec 16, 2017
@AloisJanicek
Copy link
Contributor Author

Thank you.
After 2+ weeks of using Doom, I could not identify this need for changing icon size anymore. I just tried it, it works, but even little change seems odd now - I got used to how look by default.
Nevertheless it is good that anyone now can easily change icons' size in doom if such need surface.

@hlissner
Copy link
Member

You're very welcome! This has now been merged into master, so I'll consider this resolved.

@hlissner hlissner added module:ui/modeline Pertains to Doom's :ui modeline module obsolete re:fonts Pertains to changing, using and loading fonts labels Dec 12, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 31, 2021
@hlissner hlissner modified the milestones: v2.0.8, obsolete Oct 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
is:feature Adds or requests new features, or extends existing ones module:ui/modeline Pertains to Doom's :ui modeline module re:fonts Pertains to changing, using and loading fonts status:resolved Issue was addressed internally
Projects
None yet
Development

No branches or pull requests

2 participants