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

Update 0xProto to 2.001 #1623

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update 0xProto to 2.001 #1623

wants to merge 2 commits into from

Conversation

my4ng
Copy link

@my4ng my4ng commented May 1, 2024

Description

Update 0xProto to version 2.001. This release adds an italic family as well as several fixes and improvements.

Requirements / Checklist

  • Read the Contributing Guidelines
  • Verified the license of any newly added font, glyph, or glyph set. License is: OFL-1.1 license (NO CHANGE)

What does this Pull Request (PR) do?

Update 0xProto to version 2.001.

How should this be manually tested?

fontforge --script ./font-patcher src/unpatched-fonts/0xProto/Regular/0xProto-Regular.ttf --complete --debug 2
fontforge --script ./font-patcher src/unpatched-fonts/0xProto/Italic/0xProto-Italic.ttf --complete --debug 2

@Finii
Copy link
Collaborator

Finii commented May 2, 2024

Thanks for the nice PR!

One question: Why did you put the font file into sub-directories? Is there anything in the documentation that suggests it? I have seen this now with multiple PRs.

"is a programming font focused on source code legibility."

Well, which font does not? That slogan is even lamer than before ;-) But that seems to be the official 'subtitle'.
Otoh we could add something meaningful instead, if you have any idea why it is great?


Edit:

Hmm, you ticked Read the Contributing Guidelines 🤔
Is that easy to misunderstand? I would like to improve it if needed. 🤔

@Finii Finii added this to the v3.3.0 milestone May 2, 2024
@my4ng
Copy link
Author

my4ng commented May 3, 2024

One question: Why did you put the font file into sub-directories? Is there anything in the documentation that suggests it? I have seen this now with multiple PRs.

I thought this is required in the Contributing Guidelines:

Make sure to update the correct subfolders for each font style (e.g. src/unpatched-fonts/xyz/bold/{BOLD FONT FILES HERE}), if applicable

Since 0xProto has released a new dedicated italic version, I thought it makes sense to put them in Regular and Italic folders separately.

Otoh we could add something meaningful instead, if you have any idea why it is great?

I think the features headings in the README are good pointers, especially clear differentiation and ligatures that do not deform.

@Finii
Copy link
Collaborator

Finii commented May 3, 2024

Thanks for the feedback!

if applicable

Ah I see.

are good pointers

What do you think about .... lets see

... is a programming font with clear differentiation and ligatures that do not deform

:-)


Marking this as ready, but I will hold it back until release time, and update to whatever version is current then (maybe 2.0.1 ;)

If you like maybe you can undo the sub-directory change and put both fonts into one directory. But I can change that later if you prefer.

@Finii Finii added the Ready ⛱ Ready to merge but waiting to be merged near a release label May 3, 2024
@Finii
Copy link
Collaborator

Finii commented May 3, 2024

@allcontributors please add @my4ng for code

@Finii
Copy link
Collaborator

Finii commented May 3, 2024

💭 allcontributors has some problem? I will try again later...

Copy link
Contributor

@Finii

We had trouble processing your request. Please try again later.

@my4ng
Copy link
Author

my4ng commented May 3, 2024

If you like maybe you can undo the sub-directory change and put both fonts into one directory. But I can change that later if you prefer.

Yeah I saw a couple other unpatched fonts were doing that, so I just followed suit 😅. Anyway, I have corrected that but I left the one-line description as it was.

@Finii
Copy link
Collaborator

Finii commented May 3, 2024

@allcontributors please add @my4ng for code

Copy link
Contributor

@Finii

@my4ng already contributed before to code

@Finii Finii mentioned this pull request May 3, 2024
4 tasks
@Finii Finii mentioned this pull request May 10, 2024
3 tasks
@my4ng
Copy link
Author

my4ng commented May 18, 2024

I have updated 0xProto to 2.001

@my4ng my4ng changed the title Update 0xProto to 2.000 Update 0xProto to 2.001 May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❲Aa❳ update font version bump Ready ⛱ Ready to merge but waiting to be merged near a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants