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.

Sign up for GitHub

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

Jump to bottom

Feature Request: Clearer curly braces #525

Closed
jsit opened this issue Jul 1, 2021 · 2 comments · Fixed by #587
Closed

Feature Request: Clearer curly braces #525

jsit opened this issue Jul 1, 2021 · 2 comments · Fixed by #587
Labels
Issue-Feature

Comments

@jsit
Copy link

jsit commented Jul 1, 2021

Description of the new feature/enhancement (with images if possible)

The shape of curly braces is not well-defined at small sizes and can resemble a parenthesis. Here it is in SemiLight 12 in iTerm 2:

braces-cascadia

For comparison, here's Menlo:

braces-menlo

And IBM Plex Mono:

braces-ibm-plex-mono

Proposed technical implementation details (optional)

Make the shape of the curly brace more pronounced so that it's more legible at small sizes.
@nx-lr
Copy link

nx-lr commented Oct 16, 2021 •
edited
Loading

Some more designs from Operator Mono, the braces can be a lot more wider and more angular as they looked too thin, especially as compared to ( or [.

image

@SharakPL
Copy link
Contributor

SharakPL commented Oct 17, 2021 •
edited
Loading

Honestly I like it the way it is now. The difference between { ( [ is very clear:

obraz

Maybe nose could be little longer ;)

@aaronbell aaronbell mentioned this issue Oct 22, 2021
Merged
18 tasks
DHowett pushed a commit that referenced this issue Oct 29, 2021
@aaronbell
Spooky October Bugfix (#587)
8881170 
This is a fairly comprehensive (and spooky!) 🐛💀 update resolving many
open issues.

### Arabic bugfixes
- [x] Closes #532 👻 - Additional positional variants added
- [x] Closes #535 🍂 - Corrected hamza form
- [x] Closes #540 🎃 - Dot arrangement corrected
- [x] Closes #541 🧹 - Was due to the use of anchors on those glyphs.
  These have been removed so the glyph can render as spacing.
- [x] Closes #542 🌕 - This was partly due to a [bug in Harfbuzz]. It
  has been resolved both on the font side (through a different
  implementation) and in Harfbuzz. 
- [x] Closes #549 🦸‍♀️ - Design corrected
- [x] Closes #555 💀 - All letter glyphs removed from Arabic
  Presentation form unicode slots to avoid situations where the glyphs
  are not behaving as expected.
- [x] Related to #543 - uni0615 removed as Cascadia Arabic not intended
  to support Quranic

### Other bug fixes
- [x] Closes #488 🔪 - Finally made the www ligature have the proper
  number of `w`s. 
- [x] Closes #436 🧟‍♀️ - Extended length of Powerline 'caps' to
  avoid situations where rounding can prevent overlap. This may cause
  problems if the caps are used next to one another, but that seems an
  unlikely scenario given what I've reviewed of Powerline styles. 
- [x] Closes #521 🤖 - enlarged the size of the grave character to make
  it more recognizable / legible in code. 
- [x] Closes #524 ☠️ - Added some more differentiation in stroke, and
  also created more space using hinting. 
- [x] Closes #525 🧙‍♂️ - tweaked the braces to be more twisty and
  create better differentiation from the parens. 
- [x] Closes #529 🧛‍♀️ - Changed year :P
- [x] Closes #546 👹 - ij no longer masquerading as a mark. 
- [x] Closes #563 🧟‍♂️ - corrected `locl` feature for proper
  Serbian rendering
- [x] Closes #571 🦹‍♀️ - corrected overshoot
- [x] Closes #572 🕷 - ratio symbol added
- [x] Closes #577 🍁 - shifted heights of box drawing lines to better
  align with block glyphs. Will reduce risk of non-joining forms under
  certain conditions. 

[bug in harfbuzz]: harfbuzz/harfbuzz#3069 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Issue-Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Spooky October Bugfix aaronbell/cascadia-code
3 participants
@jsit @SharakPL @nx-lr