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
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to