-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Update Cascadia Code to 2102.25 #601
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
Conversation
|
Thanks for updating this, it's appreciated.
Yeah that is wrong. It doesn't seem to use the correct RFN same name but the Microsoft one. I can take a look..
Hmm, even if you ware on Windows you should be able to produce the non windows specific versions. Can you tell me if you had some specific trouble doing so?
Seems reasonable but there are some specific adjustments the patcher makes if it's Windows vs not. So I think this is wrong sorry. Can you see my question above about the trouble you had in generating the patched fonts on Windows? Thanks! |
|
last version: v2105.24 |
|
Looking forward to any updates here 😚 |
|
It may be worth bumping this further, to 2105.24, to also include Cascadia 'Curve' (the cursive variant). |
|
I'll look into this when I have some spare time. |
|
Any update on this? @nezorflame |
|
Temporary solution could be to use https://github.com/adam7/delugia-code which patches within hours after a new Cascadia Code has been released. This is done with the nerd-font tools. |
|
Now that #681 is merged the mono fonts should no longer be "messed up". @nezorflame if you have time would you like to pull in the latest master and try re-patching? Otherwise feel free to close the PR itself and start fresh. If you don't have the time are you okay if we close the PR? Apologize for getting back into things late, it has caused PRs to languish. |
|
It's OK, I planned to do this earlier but didn't have time in the end.
I'd like to finish this PR so I'll rebase from master and fix the issues in
the upcoming week.
Best regards,
Danilkin Ilya
…_____________________
С уважением,
Данилкин Илья
Ryan L McIntyre ***@***.***> 19 ноября 2021 г. 08:33:49 написал:
Now that #681 is merged the mono fonts should no longer be "messed up".
@nezorflame if you have time would you like to pull in the latest master
and try re-patching? Otherwise feel free to close the PR itself and start
fresh.
If you don't have the time are you okay if we close the PR?
Apologize for getting back into things late, it has caused PRs to languish.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
|
Possibly it is also good to update the
|
|
@Finii Delugia looks awesome! Just wondering how are the fonts licensed? I see that the code is licensed under MIT License, but the fonts still contain (incorrect?) license info from Cascadia Code. |
The MIT License just affects the scripts, that that repo consists of. The generated font files hold the MS licensing information as required by the license that MS gives us with Cascadia Code, at least this is how I read the https://github.com/microsoft/cascadia-code/blob/main/LICENSE:
Form 2 I get the MS copyright needs to be kept and from 3 that the font needs to be renamed (of course). |
If you tried some NerdFont fonts that have this issue on Windows, please raise an issue with the names of that fonts and possible a screenshot. I have a fix and will file a MR if there is an issue :-) I just do not know if there is more than one font affected and because I'm no windows user experimenting on Windows is always a pain for me (setting up a machine, etc pp). (The commit itself is ready, but I myself did not put enough energy into researching the broadness of the issue, which I feel would be needed before patching |
The font with issue is not yet merged into the repo :-P once it's in I'll do. |
|
Yesterday Cascadia released 2111.01 🙄 Well, with no changes for us (only VF has been touched). But 2110.31 is should be. Just rambling ;) |
|
The PR above does not include |
|
Hi. I've finally got some time I can dedicate to opensource, so will probably update this PR or create a new one. |
|
@nezorflame Did you notice #723? The patched-by-723 fonts can be grabbed here |
|
@Finii yeah, just saw it, will probably create a new one then. Thanks! |
|
We updated master to 2111.01 now. This is obsolete. |
Description
Requirements / Checklist
Scripts execute without error (if necessary)What does this Pull Request (PR) do?
Updated the Cascadia Code font with the most recent (v2102.25) version of it (since the version in the repo doesn't have Cyrillic symbols which I need).
How should this be manually tested?
Works on Win10, need to check on other OSes.
Any background context you can provide?
Since this is my first PR and I'm on WIn10, I haven't found a way to produce non-Windows Compatible version of the font (yet), so I just copied the Windows Compatible patched font and renamed into the simple one (so the simple one and the Windows Compatible versions are identical). If that's inappropriate I'd like to know how to fix this.
Another thing: I might have messed the Mono-fonts - they're named differently it seems (check the last screenshot). Not sure how to fix this (just rename?)...
What are the relevant tickets (if any)?
None.
Screenshots (if appropriate or helpful)
Some examples:
Mono font: