<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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D22083#491299" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22083#491299</a>, <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>code issues are fixed.<br />
 Now for removing the further margin around the whole window:<br />
 should that be done here or in breeze?<br />
 if done here, it may make it look bad on all other widget styles?</p></div>
</blockquote>

<p>Good question. Either way is fine with me.<br />
You have more flexibility if you fix it in the widget than in the style (for instance if you change the margins of the layout in the style, this will propagate to sublayouts, and you might have a hard time to change the sublayouts back), but then you might need to test on the widget style name to not break other themes.<br />
When changing on the style side, you need a check on the widget type.<br />
Also, the best place to change on the style side is probably in ::pixelMetrics (PM_LayoutLeftMargin and the likes), rather than hacking contentsMargins directly in ::polish<br />
In either case this probably need some experimenting ...</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>