<table><tr><td style="">marten created this revision.<br />marten added a reviewer: Plasma.<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/D8617" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is the small dialogue that is summoned by the "More Preview Options" button on the "Icons" page.  Previously:</p>

<p><a href="https://phabricator.kde.org/F5466090" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5466090: folderview-properties-preview_BEFORE.png</a></p>

<p>there was no margin around either the scrolling list of the dialogue buttons.  The revised layout:</p>

<p><a href="https://phabricator.kde.org/F5466093" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5466093: folderview-properties-preview_AFTER.png</a></p>

<p>has margins added where appropriate.</p>

<p>The background colour of the scroll view is forced to the standard system background.  This does not appear to be an issue with the Breeze style, but with older styles the scroll view background seems to be use the window background colour rather than the configured view background colour.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built plasma-desktop with this change, checked appearance and operation of the configuration dialogue.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8617" rel="noreferrer">https://phabricator.kde.org/D8617</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderItemPreviewPluginsDialog.qml</div></div></div><br /><div><strong>To: </strong>marten, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>