Review Request: Mouse wheel interaction with breadcrumb-style address bar

[Peter Penz]

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


Thanks Todd for the patch. I think the "iterate through directories"-feature with the mouse-wheel is very useful. I'm unsure regarding the "go-up" feature. I'd consider this as contra productive: assuming your mouse is focused above the viewport and you scroll left/right; a minor moving of the mouse upwards to the URL navigator will result in (most probably unintended) going up some directories... So IMO this part of the feature should be skipped.

I'm not 100 % happy that currently there is quite a lot of code needed for this kind of feature (but this is not your fault). The code overlaps a lot with the "show the subdirectories as list" feature. For KDE 4.5 I've planned to do some cleanups in the KUrlNavigator & Co (the current code is still quite 1:1 to an early KDE 4.0 version that has been ported from early days of Dolphin code). One minor detail is that KUrlNavigatorButton won't have any dependency anymore to KUrlNavigator (this cyclic dependency is not nice and makes it unclear which class is responsible for what task).

So I'd suggest that we "park" this feature until I did some internal refactoring for the KUrlNavigator classes. I'd be open to port your patch then to the refactored code. Would this be OK for you?

- Peter


On 2009-12-06 08:11:18, Todd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2330/
> -----------------------------------------------------------
> 
> (Updated 2009-12-06 08:11:18)
> 
> 
> Review request for Dolphin and kdelibs.
> 
> 
> Summary
> -------
> 
> This patch allows mouse wheel interaction with the breadcrumb (not editable) address bar used in programs like Dolphin.  Currently this version of the address bar does not have any mouse wheel interaction, so this patch does not interfere with existing functionality.  
> 
> There are two different interactions supported:  
> 
> First, when the vertical mouse wheel (any normal mouse wheel) is used on one of the folders, it moves through the folders in the same level in alphabetical order.  So say you have a folder in your Home directory named "test", with 5 sub-folders, folder1, folder2, folder3, folder4, and folder5.  You are currently in folder3.  If you put your mouse over the folder3 entry in the address bar and rotate your mouse wheel down by one notch, you will switch to folder4.  If you rotate your mouse wheel up by one notch, you will move to folder2.  Move down and up by 2 (or more, in this case) notches moves you to folder5 and folder1, respectively.  While in any of these folders, using your mouse wheel on the "test" folder entry will cycle through the other subfolders in your Home directory (since those folders are at the same level as test).  When you reach the first or last folder in the directory further mouse wheel activity in that direction does nothing.
> 
> The second functionality is provided when doing a horizontal scroll (generally alt+wheel) anywhere on the breadcrumb-style address bar.  In this case, rotating the wheel by one notch in either direction moves you up one directory.  In other words, holding alt and rotating the mouse wheel by one notch is equivalent to hitting the "Up" toolbar button once.  
> 
> This patch does not change the mouse wheel behavior of the traditional text-based (editable) address bar.
> 
> I know this won't make it in before 4.5.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kfile/kurlnavigator.h 1058639 
>   /trunk/KDE/kdelibs/kfile/kurlnavigator.cpp 1058639 
>   /trunk/KDE/kdelibs/kfile/kurlnavigatorbutton.cpp 1058639 
>   /trunk/KDE/kdelibs/kfile/kurlnavigatorbutton_p.h 1058639 
> 
> Diff: http://reviewboard.kde.org/r/2330/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Todd
> 
>