<table><tr><td style="">ngraham added inline comments.
</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/D8441" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D8441#inline-36263" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">guoyunhe</span> wrote in <span style="color: #4b4d51; font-weight: bold;">config.qml:161</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Ah, I have never thought about this. Is this use case common? I have never seen people use transparent image for wallpaper. Usually it is some photos or JPG images downloaded from internet.</p>

<p style="padding: 0; margin: 8px;">This brings another problem. Blur image edge filling doesn't work well with transparent images. For transparent image you always need a background color. I am lost...</p>

<p style="padding: 0; margin: 8px;">If we say, we only support opaque image, the solution is much simpler.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I suppose we could also show the radio buttons if we detect that the image has transparency. But yeah, it would be nice if we can only support opaque images. It also hadn't occurred to me that anyone would use an image with transparency, and I don't know if I've ever seen such a thing in the wild. But of course users can be surprising! :)</p></div></div></div></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/D8441" rel="noreferrer">https://phabricator.kde.org/D8441</a></div></div><br /><div><strong>To: </strong>guoyunhe, ngraham<br /><strong>Cc: </strong>ngraham, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>