<table><tr><td style="">hein created this revision.<br />hein added reviewers: Kirigami, mart.<br />Restricted Application added projects: Plasma, Kirigami.<br />Restricted Application added a subscriber: plasma-devel.<br />hein requested review of this revision.
</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/D12102">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Changed the overall design from two lists to one list with a modal sheet to add more languages.</li>
<li class="remarkup-list-item">Replaced a modal "You need to relogin for changes" dialog with a MessageType.Positive InlineMessage.</li>
<li class="remarkup-list-item">Reworked the way missing languages are handled: The old KCM silently rewrote config and showed a warning. The new design shows an informative warning and removes the missing languages on the next save. Until then they're flagged as missing in the list.</li>
<li class="remarkup-list-item">Manages Apply button state correctly (or rather at all ...).</li>
</ul>

<p>This depends on <a href="https://phabricator.kde.org/D12097" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12097</a>.</p>

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

<p>This is currently not final code. It's a WIP upload to give Marco<br />
something to work with to fix various Kirigami and SimpleKCM problems.</p>

<p>Currently known issues:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Does not save (code is from old KCM, might have been broken there)</li>
<li class="remarkup-list-item">Disabled SwipeListItem actions do not show disabled</li>
<li class="remarkup-list-item">Placement of actions button in SwipeListItem is wonky if the contentItem is a RowLayout</li>
<li class="remarkup-list-item">SwipeListItem spews errors about positionAnimation after using an action</li>
<li class="remarkup-list-item">SwipeListItem is awkward to use, we need a drag-reorderable list delegate</li>
<li class="remarkup-list-item">OverlaySheet spews numerous warnings about not being able to find applicationWindow and activeFocusItem</li>
<li class="remarkup-list-item">The sheet is parented to the SimpleKCM's parent since there's no app window to be modal too</li>
<li class="remarkup-list-item">The footer inside an OverlaySheet sometimes moves up above the content instead of staying down</li>
<li class="remarkup-list-item">Even though SimpleKCM is just a Kirigami.ScrollablePage like Kirigami Gallery pages, an InlineMessage that fills the page width gets cut off on the left and right, so wonky code to insert margins next to them</li>
<li class="remarkup-list-item">List has window bg color as background instead of view background color</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12102">https://phabricator.kde.org/D12102</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/CMakeLists.txt<br />
kcms/translations/CMakeLists.txt<br />
kcms/translations/Messages.sh<br />
kcms/translations/kcm_translations.desktop<br />
kcms/translations/kcmtranslations.cpp<br />
kcms/translations/kcmtranslations.h<br />
kcms/translations/kcmtranslationswidget.ui<br />
kcms/translations/package/contents/ui/main.qml<br />
kcms/translations/package/metadata.desktop<br />
kcms/translations/translations.cpp<br />
kcms/translations/translations.desktop<br />
kcms/translations/translations.h<br />
kcms/translations/translationsmodel.cpp<br />
kcms/translations/translationsmodel.h</div></div></div><br /><div><strong>To: </strong>hein, Kirigami, mart<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, davidedmundson, mart, hein<br /></div>