<table><tr><td style="">asemke 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/D23119">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/D23119#inline-132013">View Inline</a><span style="color: #4b4d51; font-weight: bold;">yurchor</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kxmlgui_unittest.cpp:1089</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">AFAIK this is broken on Qt level. Cf. calendar widget in KOrganizer (Ukrainian)</p>

<p style="padding: 0; margin: 8px;"><a href="https://phabricator.kde.org/F7273188" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7273188: cal.png</a></p>

<p style="padding: 0; margin: 8px;"><a href="https://bugs.kde.org/show_bug.cgi?id=256952" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=256952</a></p>

<p style="padding: 0; margin: 8px;">There are new specifiers for the primary and alternative month names in glibc but Qt does not implement them (must be something like "LLLL YYYY" instead of "MMMM YYYY" in formatting). Rafal Luzynski (the author of the glibc patch) in a private message (2018-01-25) promised me to send the patch for Qt but this has not happened yet.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This is maybe not Qt but the locale defintion files which are probably distro specific. I just checked on SLES and on openSuse.</p>

<p style="padding: 0; margin: 8px;">for ukrainian:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">LC_TIME=uk_UA.UTF-8 locale mon
січень;лютий;березень;квітень;травень;червень;липень;серпень;вересень;жовтень;листопад;грудень</pre></div>

<p style="padding: 0; margin: 8px;">for russian:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">LC_TIME=ru_RU.UTF-8 locale mon
Январь;Февраль;Март;Апрель;Май;Июнь;Июль;Август;Сентябрь;Октябрь;Ноябрь;Декабрь</pre></div>

<p style="padding: 0; margin: 8px;">This is, except of capital letters for ru_RU, correct.</p>

<p style="padding: 0; margin: 8px;">On Mac I get however</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">січня;лютого;березня;квітня;травня;червня;липня;серпня;вересня;жовтня;листопада;грудня</pre></div>

<p style="padding: 0; margin: 8px;">and</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">января;февраля;марта;апреля;мая;июня;июля;августа;сентября;октября;ноября;декабря</pre></div>

<p style="padding: 0; margin: 8px;">which is possessive case and is wrong if you reffer to the name of the month itself.</p>

<p style="padding: 0; margin: 8px;"><a href="https://phabricator.kde.org/p/aacid/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@aacid</a> on your distribution you'll most probably get the same output as on my Mac. Under the assumption that Qt evaluates the locale definition files, your test will probably fail on other distributions which do this correctly.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23119">https://phabricator.kde.org/D23119</a></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>yurchor, apol, kde-frameworks-devel, asemke, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>