mate-desktop / mate-themes

Official themes for the MATE desktop
https://mate-desktop.org
GNU Lesser General Public License v2.1
75 stars 45 forks source link

Write theme names in title case #287

Closed rbuj closed 3 years ago

rbuj commented 3 years ago

Screenshot at 2021-02-09 17-04-26

lukefromdc commented 3 years ago

This actually renames all themes, I can see many potential issues affecting users with customized themes from this

rbuj commented 3 years ago

It should only affect the display name, as the theme name is in the path to the index.theme file.

raveit65 commented 3 years ago

I do not see any reason why we should rename the displayed name of a theme. When eg. BlueMenta is the name of a theme than BlueMenta is the name. I mean, what do think when some one wants rename Robert in Ro and Bert :) I gave my baby the name (Blue)Menta, why should it get a new name with 6 years ?

:-) :-) :-) :-) :-) :-) :-) :-)

rbuj commented 3 years ago

It's a localized string, and it only affects to English text (en_US/C locale).

There are two themes which don't use the camel case, High Contrast and High Contrast Inverse. They use title case.

rbuj commented 3 years ago

Ro Bert makes more sense if you speak German. ;-D

The name Robert is an ancient Germanic given name, from Proto-Germanic Hrōþi- "fame" and berhta- "bright" (Hrōþiberhtaz).

https://en.wikipedia.org/wiki/Robert

raveit65 commented 3 years ago

Ok, that works and it is fine for the GUI, but i see a naming mismatch with gsettings keys which we use. Eg.

[rave@mother ~]$ gsettings get org.mate.interface gtk-theme
'BlueMenta'
[rave@mother ~]$ gsettings get org.mate.Marco.general theme
'BlueMenta'

So a different naming can be confusing for users.

rbuj commented 3 years ago

This key refers to the name shown in the dialogs, which is localized:

$ grep Name /usr/share/themes/BlueMenta/index.theme 
Name[ca]=Menta blau
Name[cs]=Modrá Menta
Name[da]=BlueMenta
Name[es]=Menta azul
Name[fr]=BlueMenta
Name[gl]=AzulMenta
Name[hu]=Kék menta
Name[hy]=Կապույտ-Menta
Name[id]=BlueMenta
Name[it]=BlueMenta
Name[kab]=MentaAnili
Name[lt]=Mėlyna Menta
Name[ms]=BlueMenta
Name[nb]=BlueMenta
Name[nl]=BlauwMint
Name[oc]=BlueMenta
Name[pl]=BlueMenta
Name[ru]=Menta (Синяя)
Name[sr]=Плава мента
Name[sv]=BlueMenta
Name[tr]=MaviMenta
Name[uk]=Блакитна М'ята
Name[zh_CN]=薄荷蓝
Name[zh_TW]=BlueMenta
Name=BlueMenta
rbuj commented 3 years ago

Catalan screenshot: LANG=ca_ES.UTF-8 mate-appearance-properties

Screenshot at 2021-04-25 20-24-15

Note Catalan language doesn't use camel or title case such as English language.

raveit65 commented 3 years ago

What dos this have to do with my concern? We have different naming in GUI and gsettings key which can confuse users.

raveit65 commented 3 years ago

I don't see any reason to rename themes in GUI. It makes only a problem. I disagree with PR.

rbuj commented 3 years ago

I think you are confused with the Name, GtkTheme and MetacityTheme keys, since Name has a localitzed value unlike the other two keys.

raveit65 commented 3 years ago

....don't tell other people that they are confused. You are the one who don't want to understand.

rbuj commented 3 years ago

I think you are the one who don't want to listen... and please check the actual purpose of this key.

The Name key refers to the descriptive name displayed to the user [1], for this reason it has a localized value, that is to say it can be translated, and the translators translate it into their languages bearing in mind that it's a common noun and not a proper noun.

[1] https://specifications.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html

short name of the icon theme, used in e.g. lists when selecting themes.

rbuj commented 3 years ago

https://docs.oracle.com/cd/E19957-01/817-5310/6mkpbn3tl/index.html

raveit65 commented 3 years ago

As author of most themes i do not agree! I will revert it when you merge it.

rbuj commented 3 years ago

ok, don't worry

raveit65 commented 3 years ago

Honestly, i do not really understand why you ignore the opinion of 2 other team members. But i will try to explain it with a picture again. Bildschirmfoto zu 2021-04-26 17-54-19 As you can see m-a-p use you PR with new names, but the gsettings key use directory name from /usr/share/themes/BlueMenta. For me this is a naming mismatch which can confuse users. Keep in minds that distro maintainers use the gsettings key for gsettings overrides. For this reason i am against a renaming. It is normal in a team that not all people do agree with every idea. Not a reason to close all other PRs....

rbuj commented 3 years ago

ok, don't worry

rbuj commented 3 years ago

It is normal in a team that not all people do agree with every idea. Not a reason to close all other PRs....

I've gotten too involved in this project and maybe it’s better to take a break. I don't want my break could affect other developers. For this reason, I closed my pending contributions, because I'll probably take a long time to give feedback or apply the suggested changes. I hope you understand it. I ❤️ MATE. Yours truly, Robert.

raveit65 commented 3 years ago

You're welcome.