<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D7047" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>It occurs to me that only the "Scaled, Keep Proportions" and "Centered" positioning modes require any kind of background effect or color. With other positioning modes, the existing Background Color chooser is completely superfluous, and adding additional background effects only further exacerbates this UI disconnect when using other modes.</p>

<p>I have an idea: Add a new two-item set of Radio Buttons, and use that to choose whether the background is a solid color or your new blur feature (which is very cool), and only show this new Radio Button control if the user has selected the "Scaled, Keep Proportions" or "Centered" positioning mode.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7047" rel="noreferrer">https://phabricator.kde.org/D7047</a></div></div><br /><div><strong>To: </strong>guoyunhe, Plasma: Workspaces, ngraham<br /><strong>Cc: </strong>ngraham, broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>