<table><tr><td style="">mart accepted this revision.<br />mart added a comment.<br />This revision is now accepted and ready to land.<br />Restricted Application edited subscribers, added: kde-frameworks-devel; removed: Frameworks.
</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/D10848">View Revision</a></tr></table><br /><div><div><p>good to go, modulo that possible little code comment more</p></div></div><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/D10848#inline-52081">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kossebau</span> wrote in <span style="color: #4b4d51; font-weight: bold;">metadata.desktop:88</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Ideally the kapptemplate system would have support for an organization domain parameter which then could be used here (on my notes for some kapptemplatev2).</p>

<p style="padding: 0; margin: 8px;">For now this brush-over is proposed with KDE community contributors in mind, to help them getting applets properly prepared e.g. with integration into KDE translations right from the begin.</p>

<p style="padding: 0; margin: 8px;">Any 3rd-party which wants to keep their plasmoid 3rd-party would need to adapt the org namespace as desired, which I would expect them to do anyway if they are proud of their org.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">maybe add a # comment on top of that line staing this?</p></div></div><br /><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/D10848#inline-52080">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kossebau</span> wrote in <span style="color: #4b4d51; font-weight: bold;">%{APPNAMELC}plugin.cpp:33</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Yes, not completly sure about this one.<br />
But having seen people not getting translation setup correctly, it felt better to add the include and ki18n linking from the start, as removing those lines is easier then adding them correctly.<br />
So I would make an exception here for ki18n. But maintainers call, just proposing.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">yeah, i agree that i18n is kinda "special" and can make sense to be kept</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>brushovertemplates</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10848">https://phabricator.kde.org/D10848</a></div></div><br /><div><strong>To: </strong>kossebau, mart<br /><strong>Cc: </strong>kde-frameworks-devel, apol, michaelh, ngraham, bruns, Frameworks<br /></div>