<table><tr><td style="">anmolgautam 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/D14513">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/D14513#inline-78033">View Inline</a><span style="color: #4b4d51; font-weight: bold;">drosca</span> wrote in <span style="color: #4b4d51; font-weight: bold;">qmli18n.cpp:26</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Something like <tt style="background: #ebebeb; font-size: 13px;">setlocale(QLocale::system().name())</tt></p>

<p style="padding: 0; margin: 8px;">Yes, contextProperty is fine if it works.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This can be similar to qsTr in Qt source, something like - <br />
engine->globalObject().setProperty("i18n", engine->evaluate("function(string) { return i18n.i18n(string) }"));<br />
Here i18n is the context property (QObject corresponding to QmlI18n).<br />
But It has problems -</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">i18n as contextProperty object</li>
<li class="remarkup-list-item">It must be added to the engine at time of its creation i.e. remove singleton of i18n from QmlPlugins and move i18n to engine global object when the engine is created in QmlPluginLoader.</li>
</ol>

<p style="padding: 0; margin: 8px;">Should I change I18n so that i18n adds to global namespace or leave it now? Or is there some better solution</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R875 Falkon</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14513">https://phabricator.kde.org/D14513</a></div></div><br /><div><strong>To: </strong>anmolgautam, drosca<br /><strong>Cc: </strong>falkon, herrold, anmolgautam, SGOrava, iodelay, spoorun, ptabis, navarromorales, cochise, clivej, ach<br /></div>