Skip to content
This repository has been archived by the owner on May 3, 2021. It is now read-only.

Wrong capitalization in the German prefixes #10

Open
hsiktas opened this issue Jan 4, 2019 · 5 comments · May be fixed by #22
Open

Wrong capitalization in the German prefixes #10

hsiktas opened this issue Jan 4, 2019 · 5 comments · May be fixed by #22
Assignees
Labels
enhancement New feature or request

Comments

@hsiktas
Copy link

hsiktas commented Jan 4, 2019

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:

<string-array name="PrefixesDE">
<item>kurz nach</item>
<item>bald viertel nach</item>
<item>fast viertel nach</item>
<item>viertel nach</item>
<item>bald halb</item>
<item>kurz vor halb</item>
<item>kurz nach halb</item>
<item>bald viertel vor</item>
<item>fast viertel vor</item>
<item>viertel vor</item>
<item>bald</item>
<item>fast</item>

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. 😏

@LayoutXML
Copy link
Owner

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:

  1. All words title case - nouns are capitalized ✔️
  2. All uppercase - nouns are capitalized ✔️
  3. All lowercase - nouns are not cpaitalized ❌
  4. First word title case - nouns are not always capitalized ❌
  5. First word in every line title case - nouns are not always capitalized ❌

I will see what I can do.

@LayoutXML LayoutXML added the bug Something isn't working label Jan 5, 2019
@LayoutXML LayoutXML self-assigned this Jan 5, 2019
@LayoutXML LayoutXML added enhancement New feature or request and removed bug Something isn't working labels Jan 5, 2019
@augustuen
Copy link

I see. To fix it I should rewrite lots of code. Do people also capitalize nouns when writting informally?

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.

@msix81
Copy link

msix81 commented Jan 3, 2021

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?

@augustuen
Copy link

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.

augustuen added a commit to larshelo/Twelveish that referenced this issue Jan 3, 2021
@augustuen augustuen linked a pull request Jan 3, 2021 that will close this issue
@msix81
Copy link

msix81 commented Jan 4, 2021

Viertel should be capitalized, while times (like eins, zwölf, etc) shouldn't?

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants