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

ligatures do not work after square brace #388

Closed
mikucionisaau opened this issue Jan 7, 2021 · 5 comments
Closed

ligatures do not work after square brace #388

mikucionisaau opened this issue Jan 7, 2021 · 5 comments

Comments

@mikucionisaau
Copy link

I am trying to typeset expression [<=5], but the pair <= is not ligated after [, this pair is ligated fine after space or a number:
image

I do not experience this issue with other fonts (FiraCode, VictorMono).
The ligation also works fine after the parenthesis, curly braces, quotation marks -- everything seems consistent except after [.
So what is special about the square brace? Why the ligation does not work after [?

@auduchinok
Copy link
Contributor

auduchinok commented Jan 7, 2021

I guess it's F# attribute list brackets kicking in: [< and >]. They have ligatures for improved spacing between the chars.
@philippnurullin Is there some way to make operators like <= to have higher priority than these brackets?

@mikucionisaau
Copy link
Author

Interesting, it is so subtle, that I can barely see ligation in [< :-)
Alternatively, maybe all three symbols could be ligated so everyone is happy?

@philippnurullin
Copy link
Member

Yes, i will add an exception for this case in on of the next updates.

philippnurullin added a commit that referenced this issue Jan 25, 2021
- Added exclusion in `<-` ligature to look reasonable in negative arguments for C++ `<-1>` #107
- Removed `[||]` ligature to more consistence with `[|`  `|]` #353
- Added `ϖ` #369
- Added exclusion in `[<` `>]` to perform in `[<=5]` expression #388
- Fixed Powerline overlapping #395
@philippnurullin
Copy link
Member

Please test with the latest update https://github.com/JetBrains/JetBrainsMono/tree/master/fonts

@mikucionisaau
Copy link
Author

It fixes this issue. Thank you very much!

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

No branches or pull requests

3 participants