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

Add wider bracketleft and bracketright glyphs (from alt-hack) #393

Merged
merged 1 commit into from
Feb 20, 2018

Conversation

vl4dimir
Copy link
Contributor

@vl4dimir vl4dimir commented Feb 11, 2018

@chrissimpkins Here you go sir. 🎩

@chrissimpkins edits below ---

TODO:

  • review shapes in source
  • review hints

@chrissimpkins
Copy link
Member

chrissimpkins commented Feb 11, 2018

👍 ty! will try to get to work on this sometime in the upcoming week. Finishing the open contour PR work and this will be next up

@chrissimpkins chrissimpkins added this to the v3.003 milestone Feb 17, 2018
@chrissimpkins chrissimpkins self-assigned this Feb 17, 2018
@chrissimpkins
Copy link
Member

Getting started on this

@chrissimpkins
Copy link
Member

These changes look fantastic Vladimir. I have been examining the shapes in various bodies of source across different idioms used. Examples incoming as I have them.

@chrissimpkins
Copy link
Member

In C:

r6ufv-image

@chrissimpkins
Copy link
Member

In Go:

ht39n-image

@chrissimpkins
Copy link
Member

chrissimpkins commented Feb 20, 2018

In Python:

bc6xh-image

j8bux-image

@chrissimpkins
Copy link
Member

In Ruby:

gj94x-image

@chrissimpkins
Copy link
Member

In JavaScript:

ilf00-image

@chrissimpkins
Copy link
Member

I reviewed across sizes 8 - 14. We seem to be in good shape. Will have a look at the hinting this week and confirm that we don't need to do anything with the changes. I don't expect this to be the case. Let me know if you feel that we need any further modifications. If not and hints are ok, will merge to dev branch this week and we will plan to release this as part of the upcoming v3.003 changes. Nice work here Vladimir. This contribution is much appreciated.

@vl4dimir
Copy link
Contributor Author

Thank you! 😄

I took a couple of screenshots on Windows (Visual Studio Code) since the rendering seems to be a bit different there:

windows_8
windows_9
windows_10
windows_11
windows_12
windows_14

The brackets look good IMHO. What I see as a potential future improvement are actually the parentheses, as they now look a bit subdued when compared to curly and square brackets. Here's an exaggerated example (I added a couple of random symbols to the C example):

parens

The parentheses might use a bit more curvature, and perhaps a bit more vertical presence? For example, check out that lonely left parenthesis after hashcpy, it is a bit dominated by the brackets afterward. What do you think?

@chrissimpkins
Copy link
Member

chrissimpkins commented Feb 20, 2018

check out that lonely left parenthesis

😆 Yes, completely agree. There has been an open issue report for this for some time...

#120

I increased the spacing adjacent to the contents of the parentheses a couple of years ago but the curves and maybe even weight need to be addressed. If you have the time/interest, we're taking PR's ;)

Thanks for all of the additional images. I will review the changes on Windows and Linux as well. These shapes should not lead to hinting problems. I really like the appearance of the two glyphs immediately adjacent to each other with your changes.

Can you give me a one-liner of the design changes for the changelog? Increased length of top/bottom horizontal stroke and increased right sidebearing (and reduced left sidebearing) of left square bracket, vice versa for right square bracket spacing?

@vl4dimir
Copy link
Contributor Author

Increased length of top/bottom horizontal stroke and increased right sidebearing (and reduced left sidebearing) of left square bracket, vice versa for right square bracket spacing?

Actually the sidebearings have been reduced symmetrically on both ends, making the glyphs remain centered but having wider horizontal strokes.

So how about: "Increased length of square brackets' horizontal strokes (symmetrically reducing left and right sidebearing)."

@chrissimpkins
Copy link
Member

Perfect thanks

chrissimpkins added a commit to source-foundry/Hack-Test-Win-Installer that referenced this pull request Feb 20, 2018
@chrissimpkins
Copy link
Member

Here is the A/B diff on the shapes viewed on Windows:

2018-02-20 18 05 53

Definitely a much better render. The vertical stroke seems to render more clearly for some reason. It appears much sharper on my monitor. Looks really good.

@chrissimpkins
Copy link
Member

We're good here Vladimir. Merging to dev branch. This will be included in the v3.003 release. Thanks so much for these changes. These are high impact glyphs and the design change makes a big difference. 👍

@chrissimpkins chrissimpkins merged commit 57e0049 into source-foundry:dev Feb 20, 2018
@chrissimpkins
Copy link
Member

chrissimpkins commented Feb 20, 2018

Builds with these changes are available as of commit d7d629a on path build/ttf. No web font builds yet.

Versioned as Version 3.003;[19817712d]-dev

@chrissimpkins
Copy link
Member

Windows users who would like to kick the tires with these changes can find builds named Hack Test in this installer:

https://github.com/source-foundry/Hack-Test-Win-Installer/releases/tag/v1.2.98

@vl4dimir
Copy link
Contributor Author

@chrissimpkins Woohoo! Awesome! 😄🎉Thanks Chris!

chrissimpkins added a commit that referenced this pull request Feb 20, 2018
@chrissimpkins
Copy link
Member

Added to contributors list. Thanks again!

CodingMarkus pushed a commit to CodingMarkus/DockerHackFont that referenced this pull request Oct 9, 2018
CodingMarkus pushed a commit to CodingMarkus/DockerHackFont that referenced this pull request Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants