-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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 --font-weight-bold and set previous bold to 601 #24307
Conversation
Personally, I use 500 or 600 for semi-bold mostly these days. On Mac, these render sufficiently bold. 700 seems excessive. Would suggest adding a CSS variable for it |
Would you take this PR? I am open (I am not sure about how to satisfy all users, but some fonts don't work with 600) Actually |
As according to #24305 (comment), maybe we can use 610 ? |
Let's wait on response for #24305 (comment). Maybe we need to do some kind of detection and if a semi-bold is available, use that. |
You can put the variable, we can then tune later. |
0794662
to
7dc7454
Compare
Done, feel free to edit and fine tune. |
So, |
In the latest commit I used 610, what do you think about it? I guess it could satisfy most users. |
Noooo .... because the 610 is definitely slightly larger than 600 So you need to render the same text to see the difference. |
Ah, yes, it's a visual deception :D |
No particular reason indeed ... Personally I like to leave some buffer space when writing code 😁 It could be lowed to 601 if you like. |
Yeah, let's go with 601, the user already confirmed it works for them as well. |
I guess we do not need to bold them, just use normal font? |
Right, that's fine as well. |
Done. I guess there will be more places for bold removal in the UI not that it's more noticeable, but can be fixed later when discovered. |
* giteaofficial/main: default show closed actions list if all actions was closed (go-gitea#24234) [skip ci] Updated translations via Crowdin update nightly drone docker tag (go-gitea#24311) Remove org users who belong to no teams (go-gitea#24247) Fix typo in API route (go-gitea#24310) Add --font-weight-bold and set previous bold to 601 (go-gitea#24307) Mark `/templates/swagger/v1_json.tmpl` as generated file (go-gitea#24306) Improve External Wiki in Repo Header (go-gitea#24304) Unify nightly naming across binaries and docker images (go-gitea#24116) Wrap too long push mirror addresses (go-gitea#21120)
Fix go-gitea#24305 According to MDN, "bold" starts from 700, some fonts do not provide "bolding" for weight 600 https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight --------- Co-authored-by: silverwind <me@silverwind.io> Co-authored-by: Giteabot <teabot@gitea.io> # Conflicts: # web_src/js/components/DashboardRepoList.vue
Looking at it again, it does look a bit "odd" without bold, so revert the repolist change done in #24307. <img width="141" alt="image" src="https://user-images.githubusercontent.com/115237/234331813-c6e2402f-e099-43b3-aed6-46a0e24e3899.png">
Fix #24305
According to MDN, "bold" starts from 700, some fonts do not provide "bolding" for weight 600
https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight