<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, safaalfulaij.<br />broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.</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/D2532" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>While dragging a widget, it is reparented to "root" so it can be moved freely without the other applets reacting on that. However, in right-to-left mode, the toolbox is on the left side, the coordinate system of the applet parented to "root" did not match with the applets in "currentLayout".</p>

<p>This happened to work in left-to-right as there "currentLayout" was at coordinates 0,0. Instead of manually fiddling the margins into the positioning, just reparent the widget to another Item which is anchors.fill'd to the layout (and thus has its margins) and let QML figure it out.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I can now properly re-arrange widgets in a horizontal panel with</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);">plasmashell -reverse</pre></div>

<p>When the applet is over the right-most widget (Kickoff in the default case) it glitches slightly but that's still way better than placing the applet at seemingly random positions when letting go of the mouse pointer.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2532" rel="noreferrer">https://phabricator.kde.org/D2532</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/panel/contents/ui/ConfigOverlay.qml<br />
containments/panel/contents/ui/main.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, safaalfulaij<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>