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

Enabling the correct font #85

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carlo-colombo
Copy link

If I understand correctly the `emacs' version of the font is something that was used to test

If I understand correctly the `emacs' version of the font is something that was used to test
@rougier
Copy link
Owner

rougier commented Jun 7, 2021

Yes, it was due to a bug on OSX/Big Sur. By the way, you might want to have a look at the new (nano-theme that might be easier to hack.

@carlo-colombo
Copy link
Author

Thanks!

@louisdecharson
Copy link

Hello,
I think I have a related bug. I am using elegance.el and I have (set-face-font 'default "Roboto Mono Light 13") but it does not work well and the faces have a regular weight instead of the light one. Any thoughts on how to debug it (I've just switched to Big Sur and the line above used to work well) ?

@rougier
Copy link
Owner

rougier commented Jun 29, 2021

Are you using Emacs mac ? The last version fixed the problem. If you cannot upgrade, the trick is to create a font with only one weight (light) (see also rougier/elegant-emacs#27)

@louisdecharson
Copy link

@rougier Yes, I've seen your comment there, many thanks, I've used your trick and it's works like a charm !! 🙏
I am using emacs-plus@28 and I have the issue. Which version are you using ?

@rougier
Copy link
Owner

rougier commented Jun 29, 2021

I'm using emacs-mac. The problem has been fixed (see https://bitbucket.org/mituharu/emacs-mac/src/master/NEWS-mac) on big sur but somehow re-appear on Catalina. If you can locate the patch on Big Sur, maybe it can be back ported to emacs-plus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants