<table><tr><td style="">hpereiradacosta 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/D22083">View Revision</a></tr></table><br /><div><div><p>ather simple and attached. Feel free to add this or something similar to any other modification you plan to do. <a href="https://phabricator.kde.org/F6938398" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6938398: patch.diff</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Thank you very much Hugo.. that pixel was driving me mad ;)<br />
 Do you want to push it yourself or I just include it in the breeze modifications I'll do for the sidebar style?</p></blockquote>

<p>Well, it turns out that things are more complicated than anticipated. (it always is, right ?)<br />
As long as the sidebar is kept "transparent", the patch works. Now if you put back the sidebar background to its original white, it does not. Reason is that the viewport now coincides completely with the frame, and its background overlaps the left side vertical line. Result: the side vertical line is only visible as long as you keep the viewport transparent. <br />
So all in all: the patch goes in the right direction for what you intend to do ultimately but needs refinement.<br />
In the current code, the margins are (1,1,1,1). With my patch they are (0,0,0,0), and ultimately what you want is (0,0,0,1) or (0,1,0,0), depending on RTL. <br />
I'll try to investigate some more to fix this overlap issue, and then will submit a PR</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22083">https://phabricator.kde.org/D22083</a></div></div><br /><div><strong>To: </strong>mart, Plasma, Frameworks, VDG<br /><strong>Cc: </strong>hpereiradacosta, cfeck, ndavis, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns<br /></div>