<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/106365/">http://git.reviewboard.kde.org/r/106365/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 7th, 2012, 9:43 a.m., <b>Aaron J. Seigo</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">exposing the containment to all plasmoids is not a good idea. one of the main benefits of these QML/JS plasmoids is that they are far more sandboxed from the rest of the system. this also invites plasmoids to think they know how the shell works, which they can't -> it could just as easily be plasma-netbook ad plasma-desktop.
let's back up and look at the problem from the starting point: the goal is to put an entry in a context menu that people can select which adds an icon for that application to the panel or "the" desktop (whatever that means given multiple screens, per virtual desktop views, etc.)
first question: do we really need this exact feature? is drag and drop good enough? are there other ways we could offer? (e.g. in the "Add widgets" UI could there be a way to add applications as well?)
second question: if this is really, really required can it be done without opening access to Containment? e.g. could it be done using a Plasma::Service which does the right thing for the given shell?
also, i don't think missing this feature should stand in the way of the QML version replacing the C++ version in master. we need to get this merged as soon as possible so people can start using it and we can start improving problems that crop up.</pre>
</blockquote>
<p>On September 7th, 2012, 10:58 a.m., <b>Greg T</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi Aaron,
1. I really think as long as we support a context menu, we must also provide those 2 'add to xxx' entrys. Probably our users would be rather pissed off and call it a regression, if we drop them in kickoff-qml/KDE 4.10.
2. agreed, some kind of abstraction would be a great, no idea how to implement that though. You can really do a lot of creepy things with a containment...
3. I'm ok with that, but I never merged a feature branch into master before. Can you give me some hints how to do that? It's quite a bunch of code.
Actually mgraesslin created this widget, so maybe we should ask him for his opinion...</pre>
</blockquote>
<p>On September 7th, 2012, 12:02 p.m., <b>Marco Martin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">idea:
1) create a dataengine called org.kde.appshortcuts or something like that every source would represent a shortcut (even if just the desktop file name)
2) it has a service to add or remove shortcuts
3) the taskbar is connected to that engine and uses it for its own shortcuts
would mean that:
* would add shortcuts in the taskbar (and any other plasmoid connected to that) rather than directly the panel, so would need at least one connected to work
* add to desktop is not supported anymore and i think is good, too many assumption on what desktop it is and where it is ;)</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">interesting, but for clarification: the idea is to add those "app shortcuts" as "launchers" to the task manager?
But lets get kickoff into master before working on this topic :)</pre>
<br />
<p>- Greg</p>
<br />
<p>On September 7th, 2012, 10:58 a.m., Greg T wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma and Martin Gräßlin.</div>
<div>By Greg T.</div>
<p style="color: grey;"><i>Updated Sept. 7, 2012, 10:58 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Heya,
I need access to the containment in a qml plasmoid with c++ extensions. Actually I'm using that to add launcher items from the new kickoff-qml to the desktop/panel. Thats one of the last missing features of declarative kickoff, basically it would be ready now to replace the c++ version...
Well, certainly you know what I want to achieve, so feel free to suggest better alternatives :)</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plasma/scriptengines/javascript/plasmoid/appletinterface.h <span style="color: grey">(4f1201b)</span></li>
<li>plasma/scriptengines/javascript/plasmoid/appletinterface.cpp <span style="color: grey">(75dc2f0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106365/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>