Review Request 129601: Left mouse double click fires a "go up"

Mark Gaiser markg85 at gmail.com
Fri Dec 2 15:53:20 GMT 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129601/#review101220
-----------------------------------------------------------



Hi Lorenz,

Thank you for your patch!

I'm glad you made this optional and disabled by default. I guess it really depends on your workflow if one would user this feature or not.
I personally would be a bit skeptical to using it since i would likely have accidental double clicks which would give me a "huh, now i changed folders, how did that happen" ;)

- Mark Gaiser


On dec 2, 2016, 11:36 a.m., Lorenz Haas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129601/
> -----------------------------------------------------------
> 
> (Updated dec 2, 2016, 11:36 a.m.)
> 
> 
> Review request for Dolphin and KDE Usability.
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> If you double click with the left mouse besides an item, you go up the folder hierarchy.
> 
> BUG: 307505
> 
> 
> Diffs
> -----
> 
>   src/dolphinmainwindow.cpp e5103fd 
>   src/kitemviews/kitemlistcontroller.h b8a93ed 
>   src/kitemviews/kitemlistcontroller.cpp 1c86ff0 
>   src/settings/dolphin_generalsettings.kcfg c724afc 
>   src/settings/navigation/navigationsettingspage.h 5410a4e 
>   src/settings/navigation/navigationsettingspage.cpp e37a35d 
>   src/views/dolphinview.h 0b0d819 
>   src/views/dolphinview.cpp 4105628 
> 
> Diff: https://git.reviewboard.kde.org/r/129601/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lorenz Haas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20161202/861eb6d5/attachment.htm>


More information about the kfm-devel mailing list