Review Request: big revamp of Device Notifier

Aaron Seigo aseigo at kde.org
Fri Sep 11 18:25:54 CEST 2009



On 2009-09-10 18:18:27, Giulio Camuffo wrote:
> > I still don't like the fact that you have to click twice to select the action if there is only one available action.. we should find a solution for this..
> 
> Giulio Camuffo wrote:
>     I agree.
>     Maybe we could show the actions on hover. This would require then a single click, but on the other hand it could be a bit annoying if you have many devices and you have to get to the one on the other side of the applet.
> 
> Michal Dutkiewicz wrote:
>     > Maybe we could show the actions on hover.
>     
>     What about showing them with delay (so fast mouse move won't trigger showing)?
> 
> Giulio Camuffo wrote:
>     This could be a solution, but could make this feature less discoverable
> 
> Michal Dutkiewicz wrote:
>     > This could be a solution, but could make this feature less discoverable
>     
>     Depends how long is that delay. ;-)
>     I think that it could be something like 200 milliseconds, but could be adjusted after some testing.

300ms is the standard we tend to use in Plasma, and this is based on quite a bit of testing. having things show just when you move the mouse through them to get to something else is visually annoying/tiring, but you don't want to "wait". 300ms (just under a 1/3rd of a second) seems to be a fairly "magic" compromise that works well.

in any case, i need to download and try the latest version of this patch to see how it's going.... will do so today.


- Aaron


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1370/#review2266
-----------------------------------------------------------


On 2009-09-01 16:55:56, Giulio Camuffo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1370/
> -----------------------------------------------------------
> 
> (Updated 2009-09-01 16:55:56)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This is a patch that modifies quite heavily the behaviour of the Device Notifier.
> It comes from here: http://kde-look.org/content/show.php/Device+Manager?content=106051
> It can show the not removable devices too, it can mount them automatically or with a click, since the "eject" button is a "mount" button when the volume is umounted. So that guy on the dot will be ok.
> It can hide some items in the same way as Dolphin's places (hide item/ show all).
> Finally, it shows the various opening actions under the device instead of calling that xp-ish window.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.h 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/CMakeLists.txt 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurationpage.ui PRE-CREATION 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.h 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.cpp 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.h 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.cpp 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.h 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.cpp 1013960 
> 
> Diff: http://reviewboard.kde.org/r/1370/diff
> 
> 
> Testing
> -------
> 
> I'm using it every day since I released 0.1 on Kde-look. I tried all the options on my pc and they work. Some people on kde-look posted some comments about some problems, but it seems to me they are very particular cases, so in my opinion it is quite stable to go in trunk, but anyway review it! :)
> 
> 
> Screenshots
> -----------
> 
> screen
>   http://reviewboard.kde.org/r/1370/s/183/
> 
> 
> Thanks,
> 
> Giulio
> 
>



More information about the Plasma-devel mailing list