<div class="gmail_quote">On Thu, Jun 21, 2012 at 11:09 AM, Aaron J. Seigo <span dir="ltr"><<a href="mailto:aseigo@kde.org" target="_blank">aseigo@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Wednesday, June 20, 2012 23:54:57 Martin Klapetek wrote:<br>
> I imagine this could then be used in Dolphin's context menu 'Actions' and<br>
> possibly in Gwenview too. Should there be such dbus-interface, I would add<br>
> those patches to Dolphin and Gwenview.<br>
<br>
</div>the things i look at and weigh are:<br>
<br>
* will the feature work consistently (multiple screens, per-virtual desktop<br>
containments, etc..) and not just work for some configurations<br></blockquote><div><br></div><div>Very important points, this is crucial to good user experience. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


* what is the cost to the codebase (how complex will the patches to plasma-<br>
desktop be .. and this does require patches there)<br></blockquote><div><br></div><div>This however I have no idea about.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


* is this valuable enough to add another item to the context menu? context<br>
menus ought to be fairly focused otherwise they become a clutter of options<br>
that become more difficult to sort through. we support setting the wallpaper<br>
through drag and drop, so this is a consideration here.<br></blockquote><div><br></div><div>Depends. The Dolphin's "Actions" context sub-menu on my machine currently has </div><div><br></div><div>"Encrypt file"</div>

<div>"Preview file"</div><div>"Send as email attachment"</div><div>"Send file to IM contact"</div><div>"Create file project with K3B"</div><div><br></div><div>adding "Set this image as wallpaper" does not seem to bring too much clutter imho. I think it might be more useful to average users than "Encrypt file" (not suggesting to remove it).</div>

<div><br></div><div>As for d'n'd - it does not work with Gwenview when viewing images. Dragging the image actually pans the image, so there's no way to drag it outside the window. Easy "fix" is to switch to "Browse Mode" and drag it from there. But the usual use case would probably be "User opens an image. He wants to set it as a wallpaper. So he clicks a button and he has a new wallpaper." Going through the "Browse mode" is not as convenient, especially if you have to resize Gwenview so that you see the desktop and you have a place to drop it to. It makes sense for it to be in a context menu; it's what they're for after all (if they're not abused).</div>

<div><br></div><div>I think it might also come useful on other devices, where you don't have dragging capabilities - currently Active on tablets, dragging an image would(/should?) also pan the image, not drag it. Or the planned set top boxes, no dragging at all. Here the context menu may come very handy.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
given the above, i'd much rather see the DnD support that already exists and<br>
works used with dolphin and gwenview.<br></blockquote><div><br></div><div>I think that part of that effort should be making this feature more discoverable, especially for newcoming users from other OSs, where dragging a file to the desktop means "make a copy on/move it to the desktop". So the user has no idea he can set a wallpaper that way, unless told so by others.</div>

<div><br></div></div>--<div><font color="#666666">Martin Klapetek | KDE Developer</font></div><br>