-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Can you make the r
look better?
#601
Comments
It has its form for legibility reasons. Maybe I’ll return alternative form as a stylistic subset one day, thanks for the reminder! |
IMO, the current |
The "r" is not particularly in need of increased legibility. Currently it stands out like a sore thumb. My cooworker when shown Fira said hmm....looks cool...oh wait! Thats what the "r" looks like? Nope not doing it. Nevermind. I'm in the same camp. |
Noto Sans Mono seems has the same problem. |
@brilliantly-insane How about dropping the serif on the left side of the r too? |
@j-f1 Agreed; thought it might work with the |
Yup. I think the |
The r is keeping me and my colleagues from using this font. I never encountered someone that actually likes the current r in this font. |
There are a few other traditional serifs such as the i j L, however the r is unusual for an r. Perhaps it would look right if the curved width was 10% wider or we had been used to seeing it in other fonts (there may be a font out there that does have this serif? But I don't know one off the top of my head). Almost all serifs are horz (in this font), the r has the only one vertical. I mean if for the r why not for the c as well? or s or z or 7? The point is that the r stands out. The r is not harmonious. |
It isn’t. Originally r didn’t have serifs. Those were added because people complained about low legibility. r without serifs used to look like lowercase i. If you dig deep enough I guess you can find the original r in oldest Fira Mono files |
That's hard to believe really. How many people complained? Whomever I ask, they prefer using another font instead of Fira, because of the serifs in Fira r. |
Yes, important link above. Note that people immediately complained about the result in that issue. Its a real SHAME that an "issue" with the r was the resolved this way, when this issue should be an issue for many other mono-spaced fonts, yet its not? IMO: It wasn't a real issue then, and now we have an ugly r. |
I didn’t adopt Fira because of the r too, I just... can’t. That’s a pity because everything is so good but this spoils it. Maybe if I used it for several months I could get used to it, but each time I tried I just kept staring at those r letters. |
@diplomatic-dev how did you go with that? |
I think fira code is the best font for intellij and windows console! but i agree "r" looks awkward... |
Another viewpoint (possibly unpopular): To me, the However, that is not to say the glyph looks good from a useability perspective. I feel like removing only the offending serif and keeping the body in the same form as the current |
I'm mainly using |
+1, please add a sans |
I stopped seeing it after a week, now it looks just normal. The author, having looked at it for years, must think: what do these people have against a perfectly normal r? |
Am I the only one who likes the r? |
@steebchen you are not alone. I don’t believe it needs defending. |
I vote for a truly sans-serif version of Fira Code. It has become my favourite coding font, but the serifs look extremely odd, even more so on a 4k monitor with more pixels available for font rendering. Most notably, |
How do I easiest get the version of FiraCode with sans-serif |
There's a tool OpenType Feature Freezer. With it you could patch the font yourself. E.g. |
@alexeyten Thanks, I'll give that a try! |
Fira Code is the best font for writing code (programming). My only complaint is the
r
looks weird compare to other letters. Can you make ther
looks better? I mean make it as sans-serif like the other letters, instead of being the only serif letter in the group.Thank you.
The text was updated successfully, but these errors were encountered: