-
Notifications
You must be signed in to change notification settings - Fork 19
Wrong capitalization in the German prefixes #10
Comments
Hi @hsiktas , I see. To fix it I should rewrite lots of code. Do people also capitalize nouns when writting informally? At the momemnt there are 5 capitalization options:
I will see what I can do. |
To my knowledge the capitalisation of nouns in German is quite alike to capitalisation of the first word in English, meaning that there's no situation where you can not do it (according to the rules), but it's left out in casual messages. I don't feel like this is such a situation however, although I think the all uppercase and all lowercase options should be maintained. As for implementation it seems that it could easily be solved by capitalising the nouns in strings.xml, and adding a toLowerCase on the output of WordClockTask.capitalise2(). I've got a fix ready to be pull-requested. |
I love the watch face, however the wrong German capitalization looks quite unprofessional. @augustuen if you have a fix ready, why don't you raise a PR? |
I'm not quite sure why I didn't at the time. Maybe I couldn't separate it from the other modifications I'd done and promtly forgot about it? IDK, that's living with ADD I guess. I did find my fix though, will submit it in a little while when I've sorted my Gradle issues. The way I understand it though, Viertel should be capitalized, while times (like eins, zwölf, etc) shouldn't? Would love your input @msix81 @hsiktas. My german is pretty good but I've never encountered this issue before. |
Exactly - "Viertel" is a noun (engl. "quarter"), whereas numbers and other words like "halb" or "oder so" are not. First word should be capitalized as well, but that can be achieved by the current options as far as I understand. |
Hi,
this is only a cosmetic grammar issue but because it is not covered by the current capitalization options, this might count as a bug instead of a German translation error. 🤔
In German we begin all nouns with a capital letter and this applies to the word "Viertel" from the German prefixes:
Twelveish/app/src/main/res/values/strings.xml
Lines 330 to 342 in a3a1305
Here is a nice overview of how that prefix is used in the right way: https://www.dict.cc/?s=Viertel+vor
I don't know if this should be a new option for the capitalization menu of if this should be an exception for all existing capitalization options but in its current state it just looks wrong for German speakers. 😏
The text was updated successfully, but these errors were encountered: