<table><tr><td style="">aspotashev added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25530">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D25530#inline-144113">View Inline</a><span style="color: #4b4d51; font-weight: bold;">waqar</span> wrote in <span style="color: #4b4d51; font-weight: bold;">loader.cpp:255</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I used wikipedia and the description files provided with the dictionaries for this information. It may not be correct however. If you know the language or the rules, please feel free to correct me and I will amend the commit accordingly</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">There may be different consistent approaches to capitalization:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Use the spelling like if the language name comes in the middle of a sentence. This implies all-lowercase spelling for many Cyrillic-based languages, e.g. "русский" for Russian.</li>
<li class="remarkup-list-item">Use the spelling like if the language name comes in the beginning of a sentence. In many languages that means that first letter is in uppercase, e.g. "Русский" for Russian.</li>
</ol>

<p style="padding: 0; margin: 8px;">I don't know which of these approaches is taken by Qt, but we should follow the pattern. Until then, I can't say how to fix this patch.</p>

<hr class="remarkup-hr" />

<p style="padding: 0; margin: 8px;">After having a look on the Qt source code (qlocale_unix.cpp, qlocale_win.cpp) I'm not sure where do these native language names from on Unix/Linux. QSystemLocale::query() as defined in qlocale_unix.cpp seems to returns an empty QVariant.</p>

<p style="padding: 0; margin: 8px;">I think it would be really helpful if we add unit tests for this Loader::languageNameForCode() method.</p>

<hr class="remarkup-hr" />

<p style="padding: 0; margin: 8px;">I don't know these languages, however I can read Russian and at least detect capital letters in languages that mostly reuse the Russian alphabet.</p>

<p style="padding: 0; margin: 8px;">Now so sure about Chuvash language anymore because English Wikipedia says "Чӑвашла" while Russian <br />
Wikipedia and many other resources in Russian say "чӑвашла", e.g. <a href="https://en.wiktionary.org/wiki/чӑваш_чӗлхи" class="remarkup-link" target="_blank" rel="noreferrer">https://en.wiktionary.org/wiki/чӑваш_чӗлхи</a></p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R246 Sonnet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25530">https://phabricator.kde.org/D25530</a></div></div><br /><div><strong>To: </strong>waqar<br /><strong>Cc: </strong>aspotashev, aacid, yurchor, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>