<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.
</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/D20972">View Revision</a></tr></table><br /><div><div><p>In general +1, let's not let this tiny UI improvement die.</p>

<p>However the implementation is not quite finished: Whenever you set a rightMargin to make something look better next to a scrollbar, you need to conditionalize that on it being a LTR language, and add a matching version for the leftMargin when using an RTL language, because in that case, the scrollbar goes on the other side of the view. Basically like this:</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);">Layour.rightMargin: LayoutMirroring.enabled ? 0 : Kirigami.Units.smallSpacing / 2
Layout.leftMargin:  LayoutMirroring.enabled ? Kirigami.Units.smallSpacing / 2 : 0</pre></div>

<p>Alternatively if the items are inside a Layout--as they are here--instead of using margins, you can add an <tt style="background: #ebebeb; font-size: 13px;">Item</tt> to the layout with the width of the spacing you want, and RTL layouts will look fine automatically. This works because the direction of items in a Layout get automatically reversed for RTL languages, while the same is not true of manually-set margins.</p>

<p>Your choice; I think using an <tt style="background: #ebebeb; font-size: 13px;">Item</tt> is a bit more elegant but I don't object to the alternative approach.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R124 System Settings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20972">https://phabricator.kde.org/D20972</a></div></div><br /><div><strong>To: </strong>filipf, VDG, ngraham, davidedmundson<br /><strong>Cc: </strong>abetts, davidedmundson, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>