Bug 224021

Summary: zoom shortcuts inconsistent between editor and preview
Product: [Applications] digikam Reporter: Johannes Wienke <languitar>
Component: Usability-KeyboardAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 1.1.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Unspecified   
Latest Commit: Version Fixed In: 1.2.0
Sentry Crash Report:

Description Johannes Wienke 2010-01-24 12:46:48 UTC 
Version:           1.1.0 trunk (using KDE 4.3.4)
Installed from:    Archlinux Packages

The keyboard shourtcuts for zooming are inconsistent between preview and editor. In the preview widget you have to use ctrl + plus or minus, whereas in the editor simply using plus an minus is sufficient. I think this should be unified or allowing both ways in all places.
Comment 1 caulier.gilles 2010-01-24 15:14:45 UTC 
I agree, if others people want too. It's a 5 minutes fix (:=)))...

Gilles Caulier
Comment 2 caulier.gilles 2010-01-25 18:51:35 UTC 
Jahonnes, 

Same problem to fix in Light Table...

Gilles Caulier
Comment 3 Johannes Wienke 2010-01-25 19:01:29 UTC 
But which shortcut do we want? Or both?
Comment 4 caulier.gilles 2010-01-25 21:10:14 UTC 
Iconview : dolphin/konqueror use CTRL +/-
preview  : gwenview use CTRL +/-
editor   : gimp use +/- 

Note : firefox use CTRL+/- to zoom web page.

For me both must be available.

I vote for CTRL +/- as primary, and +/- as secondary shorcuts

Gilles Caulier
Comment 5 caulier.gilles 2010-01-25 21:25:37 UTC 
Note : Krita use also CTRL+/-

Gilles Caulier
Comment 6 caulier.gilles 2010-01-25 21:37:34 UTC 
A general review to keyboard shortcuts : 

http://en.wikipedia.org/wiki/Table_of_keyboard_shortcuts

.. Can be interresting for the future...

Gilles Caulier
Comment 7 caulier.gilles 2010-01-25 21:40:59 UTC 
Another link:

http://morris-photographics.com/photoshop/shortcuts

Photoshop use also CTRL +/- as primary shortcuts

Gilles Caulier
Comment 8 caulier.gilles 2010-01-29 14:33:08 UTC 
Johannes

Didi you manage this entry before 1.1.0 release ?

Gilles Caulier
Comment 9 Johannes Wienke 2010-01-29 14:46:31 UTC 
No, I thought I wait for more opinions.
Comment 10 caulier.gilles 2010-02-02 13:03:55 UTC 
SVN commit 1084057 by cgilles:

Zoom Plus/Minus shortcuts :
- Set alternate shortcuts as Plus/Minus keys.
- Primary shortcuts still default values set by KDELibs: Ctrl+ and Ctrl-.
BUGS: 224021


 M  +8 -2      digikam/digikamapp.cpp  
 M  +3 -3      digikam/digikamapp_p.h  
 M  +6 -0      utilities/cameragui/cameraui.cpp  
 M  +8 -4      utilities/imageeditor/editor/editorwindow.cpp  
 M  +9 -5      utilities/lighttable/lighttablewindow.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1084057