<div dir="ltr"><div>Hello,</div><div><br></div>On 10 October 2013 15:48, Sebastian Kügler <span dir="ltr"><<a href="mailto:sebas@kde.org" target="_blank">sebas@kde.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">What is holding back the merge from your point of view?<span class=""><font color="#888888"><br>

</font></span></blockquote></div><br>it depends from the containment. Right now `plasmoidviewer -a foo`</div><div class="gmail_extra">will use the plasmoidviewer's containment which is missing</div><div class="gmail_extra">
* more testing and someone to review the code</div><div class="gmail_extra">* no way to access the plasmoid's settings</div><div class="gmail_extra">* formfactor/location can't be changed from the view, only from the command line, like  `plasmoidviewer -a foo1 -l foo2 -f foo3`<br>
* no konsolepreviewer, ok we can live without this one :)</div><div class="gmail_extra">* the ui isn't perfect(check the attached image)</div><div class="gmail_extra"><br></div><div class="gmail_extra">*but* if we use an existing containment like `plasmoidviewer -a org.kde.example.widgetgallery -c org.kde.desktopcontainment`</div>
<div class="gmail_extra">then its mostly ok, but we are still missing</div><div class="gmail_extra">* more testing and someone to review the code</div><div class="gmail_extra"><br></div><div class="gmail_extra">also some other features are missing like to change the width and height using args, but those aren't blockers.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">So, </div><div class="gmail_extra"><br></div><div class="gmail_extra">a. we can merge the plasmoidviewer now into frameworks-scratch and change the default containment</div>
<div class="gmail_extra">in order  to provide most of the plasmoidviewer's 1 functionality and continue working on plasmoidviewer2 branch the</div><div class="gmail_extra">rest of the features.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">b. we can wait until its ready(including all the features and the plasmoidviewer's containment).</div><div class="gmail_extra"><br></div><div class="gmail_extra">I believe that option a is good enough for the time being because it does most of the job and also plasmoidviewer 1 didn't</div>
<div class="gmail_extra">have a custom containment in KDE4.</div><div class="gmail_extra"><br></div><div class="gmail_extra">a or b? :)</div><div class="gmail_extra"><br></div><div class="gmail_extra">Regards,</div><div class="gmail_extra">
Giorgos</div><div class="gmail_extra"><div><br></div>-- <br><div dir="ltr">Giorgos Tsiapaliokas (terietor)<br><br><a href="http://terietor.org" target="_blank">terietor.org</a></div>
</div></div>