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

refactor: use new icons for done/undone as per nerd fonts 3.0 #849

Merged
merged 1 commit into from
May 12, 2023

Conversation

obreitwi
Copy link
Contributor

@obreitwi obreitwi commented May 6, 2023

The icons used to conceal 'done' and 'pending' tasks were moved to new code points. This commit fixes them according to this translation table.

@vhyrro
Copy link
Member

vhyrro commented May 6, 2023

Oh dang, thanks for making us aware of this. I wonder - how long will it take before this change propagates to everyone? I'm not sure how fast the release cycle is for fonts in various package managers, moreso when you start talking about other operating systems like Mac or Windows. I'm afraid that merging this right away will cause some long term confusion, so perhaps we should wait a few days before the merge?

@obreitwi obreitwi force-pushed the fix/codepoints/task_conceal branch from 30cd6ab to 5c880d8 Compare May 6, 2023 20:16
@obreitwi
Copy link
Contributor Author

obreitwi commented May 6, 2023

Yeah, I would suspect the change will take a while to propagate. For example, the font I use was patched earlier only earlier this week.

Is there an FAQ or trouble shooting guide it could be added to? Or a compatibility setting that could be used to switch the used icon…
Because I imagine I won't be the only one that's greeted with blank rectangles instead of done/pending tasks.

Afaik, there is no way to check if a code point is present in the font currently used by the terminal, so it will require user action… :/

@obreitwi obreitwi force-pushed the fix/codepoints/task_conceal branch 2 times, most recently from b7269fb to 7be8712 Compare May 10, 2023 17:05
@obreitwi obreitwi force-pushed the fix/codepoints/task_conceal branch from 7be8712 to 43ce246 Compare May 11, 2023 22:40
@vhyrro
Copy link
Member

vhyrro commented May 12, 2023

I see quite a lot of other projects adapt to nerd fonts 3.0 so I guess it's our time to do so now too! Thanks for the PR :)

@vhyrro vhyrro changed the title Adapt icons for 'done' and 'pending' task to new nerd font code points refactor: use new icons for done/undone as per nerd fonts 3.0 May 12, 2023
@vhyrro vhyrro merged commit 4f5448c into nvim-neorg:main May 12, 2023
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.

2 participants