<br><br><div class="gmail_quote">2009/8/22 Giulio Camuffo <span dir="ltr">&lt;<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
&gt; 2009/8/21 Alessandro Diaferia &lt;alediaferia at <a href="http://gmail.com" target="_blank">gmail.com</a>&gt;<br>
&gt;<br>
&gt; &gt; 2009/8/21 Giulio Camuffo &lt;giuliocamuffo at <a href="http://gmail.com" target="_blank">gmail.com</a>&gt;<br>
<div class="im">&gt; &gt;<br>
&gt; &gt;&gt; &gt; On 2009-08-21 17:23:18, Alessandro Diaferia wrote:<br>
&gt; &gt;&gt;<br>
</div>&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifie<br>
<div class="im">&gt; &gt;&gt;r.h, line 106<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; &gt; &lt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; <a href="http://reviewboard.kde.org/r/1370/diff/3/?file=10433#file10433line106" target="_blank">http://reviewboard.kde.org/r/1370/diff/3/?file=10433#file10433line106</a>&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; &gt;     consider deleting white spaces<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; eh, the problem with white spaces is that they are... white :p they<br>
&gt; &gt;&gt; escape!<br>
&gt; &gt;<br>
&gt; &gt; That&#39;s why reviewboard shows them :p<br>
&gt; &gt;<br>
&gt; &gt;&gt; &gt; On 2009-08-21 17:23:18, Alessandro Diaferia wrote:<br>
&gt; &gt;&gt;<br>
</div>&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifie<br>
<div class="im">&gt; &gt;&gt;r.cpp, line 335<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; &gt; &lt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; <a href="http://reviewboard.kde.org/r/1370/diff/3/?file=10434#file10434line335" target="_blank">http://reviewboard.kde.org/r/1370/diff/3/?file=10434#file10434line335</a>&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; &gt;     No need for this anymore: i&#39;ve committed a fixed for the bug and<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; closed it.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; whooo! great! thanks<br>
&gt; &gt;<br>
&gt; &gt; =) no problem<br>
&gt; &gt;<br>
&gt; &gt;&gt; - Giulio<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; -----------------------------------------------------------<br>
</div><div class="im">&gt; &gt;&gt; This is an automatically generated e-mail. To reply, visit:<br>
</div><div class="im">&gt; &gt;&gt; <a href="http://reviewboard.kde.org/r/1370/#review2112" target="_blank">http://reviewboard.kde.org/r/1370/#review2112</a><br>
&gt; &gt;&gt; -----------------------------------------------------------<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; On 2009-08-21 14:05:09, Giulio Camuffo wrote:<br>
&gt; &gt;&gt; &gt; -----------------------------------------------------------<br>
</div><div class="im">&gt; &gt;&gt; &gt; This is an automatically generated e-mail. To reply, visit:<br>
&gt; &gt;&gt; &gt; <a href="http://reviewboard.kde.org/r/1370/" target="_blank">http://reviewboard.kde.org/r/1370/</a><br>
&gt; &gt;&gt; &gt; -----------------------------------------------------------<br>
&gt; &gt;&gt; &gt;<br>
</div><div class="im">&gt; &gt;&gt; &gt; (Updated 2009-08-21 14:05:09)<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Review request for Plasma.<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt;<br>
</div><div class="im">&gt; &gt;&gt; &gt; Summary<br>
&gt; &gt;&gt; &gt; -------<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; This is a patch that modifies quite heavily the behaviour of the<br>
&gt; &gt;&gt; &gt; Device<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Notifier.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; It comes from here:<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; <a href="http://kde-look.org/content/show.php/Device+Manager?content=106051" target="_blank">http://kde-look.org/content/show.php/Device+Manager?content=106051</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; It can show the not removable devices too, it can mount them<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; automatically or with a click, since the &quot;eject&quot; button is a &quot;mount&quot;<br>
&gt; &gt;&gt; button when the volume is umounted. So that guy on the dot will be ok.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; It can hide some items in the same way as Dolphin&#39;s places (hide item/<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; show all).<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; Finally, it shows the various opening actions under the device instead<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; of calling that xp-ish window.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; Diffs<br>
</div>&gt; &gt;&gt; &gt; -----<br>
<div class="im">&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/CMakeLists.tx<br>
</div>&gt; &gt;&gt;t 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configuration<br>
&gt; &gt;&gt;page.ui PRE-CREATION<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifie<br>
<div class="im">&gt; &gt;&gt;r.h 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifie<br>
&gt; &gt;&gt;r.cpp 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespacein<br>
&gt; &gt;&gt;fodelegate.h 1013960<br>
&gt; &gt;&gt;<br>
</div>&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespacein<br>
&gt; &gt;&gt;fodelegate.cpp 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialo<br>
&gt; &gt;&gt;g.h 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialo<br>
<div class="im">&gt; &gt;&gt;g.cpp 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.<br>
&gt; &gt;&gt;h 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.<br>
&gt; &gt;&gt;cpp 1013960<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; Diff: <a href="http://reviewboard.kde.org/r/1370/diff" target="_blank">http://reviewboard.kde.org/r/1370/diff</a><br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Testing<br>
&gt; &gt;&gt; &gt; -------<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; I&#39;m using it every day since I released 0.1 on Kde-look. I tried all<br>
&gt; &gt;&gt; &gt; the<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; options on my pc and they work. Some people on kde-look posted some<br>
&gt; &gt;&gt; comments about some problems, but it seems to me they are very<br>
&gt; &gt;&gt; particular cases, so in my opinion it is quite stable to go in trunk,<br>
&gt; &gt;&gt; but anyway review it! :)<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; Screenshots<br>
&gt; &gt;&gt; &gt; -----------<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; screen<br>
&gt; &gt;&gt; &gt;   <a href="http://reviewboard.kde.org/r/1370/s/183/" target="_blank">http://reviewboard.kde.org/r/1370/s/183/</a><br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Thanks,<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Giulio<br>
&gt; &gt;<br>
</div>&gt; &gt; --<br>
<div class="im">&gt; &gt; Alessandro Diaferia<br>
&gt; &gt; KDE Developer<br>
&gt;<br>
&gt; Oh one last thing that comes to my mind is that the configuration dialog<br>
&gt; does not follow GUI usability guidelines. But that shouldn&#39;t be a problem,<br>
&gt; there can be a patch for it later :)<br>
&gt;<br>
&gt; --<br>
&gt; Alessandro Diaferia<br>
&gt; KDE Developer<br>
<br>
</div>Yes, I know that I&#39;m not good at all at drawing interfaces, and I don&#39;t like<br>
very much the designer. I prefer very much to work underground :)<br>
Anyway, since I don&#39;t have an svn account, if you judge the patch ready, feel<br>
free to commit it.<br>
<div><div></div><div class="h5">_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br>I&#39;ve tested the patch and it works correctly. I&#39;d clean indentation a little before committing. Just waiting until sunday before committing so, if someone is against the patch, speak now or shut up forever =)<br clear="all">
<br>-- <br>Alessandro Diaferia<br>KDE Developer<br>