-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
Capitalize words based on GNOME HIG #191
Comments
yes, thanks in advance! |
I am again re-reading the HIG and now I am under impression what for theme options only first word should be capitalized, while every word should be capitalized only for buttons and section headlines, for example transmission: as you see in headlines and buttons (and main and dropdown menus, which are not on the screenshot) every word is capitalized while options are not |
In HIG, "Sentence capitalization" is used for checkbox and radio labels, since these labels are usually sentences. IMHO Oomox's row labels are rather closer to "Header" than "Sentence". |
headers in theme models are typed as |
Hmm, in GNOME Settings, "Header capitalization" is adapted to the Another example from GNOME's mockup: https://gitlab.gnome.org/Teams/Design/app-mockups/blob/master/tweaks/tweaks-wires.png |
i've also capitalized few more labels and replaced |
i did few more writing style and typography improvements, please install the latest oomox-git and check if it's all done |
Looks much better! Nice job! A few things I noticed:
|
|
|
|
|
please check the current status of this ticket |
So sorry for the late reply. I checked the current status now.
Regarding 2: I am sorry, but to be honest, I prefer the previous style of only the "Theme Options" section (for consistency with other themes). It was easier to see to me. Also IMO, "Primary Caret" and "Secondary Caret" should be in the top "Theme Colors" section. However, if you prefer the current style, I won't be against it.
Maybe if there are separators between rows, it may look better, I guess.
AFAIK, there is no useful widget or style for such case, unfortunately. |
|
@nana-4 so, can this one be closed or more capitalization / text stuff remains? |
Sorry, I forgot to reply this:
But it's also not always "Selection", isn't it? (e.g. progress bar) For the sake of consistency, personally I prefer:
But this is a relatively trivial thing, so if it doesn't fit your taste, please go ahead and let's close this issue. |
selected_bg / fg naming is sorta for uniformity with gtk2 and gtk3 themes there those are standard naming while, of course, in UI there is no need to follow that technical side, but still could be nice for people who hacking on theme files manually
those minor renaming i don't mind at all
this could be quite misleading because under the hood theme var is called ACCENT |
👍
Indeed. Regarding that, I take it back. |
See "Capitalization" in https://developer.gnome.org/hig/stable/writing-style.html.en
@actionless Do you agree with this? (e.g. "Theme style" -> "Theme Style") If you agree, should I send a PR for this?
The text was updated successfully, but these errors were encountered: