Bug 145558

Summary: in menu view: window size/original size: ctrl-shift-z obsolete?
Product: [Applications] digikam Reporter: Daniel Bauer <linux>
Component: Usability-KeyboardAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In: 0.9.2

Description Daniel Bauer 2007-05-17 16:23:20 UTC
Version:           0.9.2svn (using KDE KDE 3.5.6)
Installed from:    SuSE RPMs
OS:                Linux

In menu "view" ("Ansicht") you can choose "original size" (ctrl-shift-z) and "zoom to window size" (ctrl-shift-a).

But in fact ctrl-shift-a acts as a toggle between window size and original size, but ctrl-shift-z does nothing. Also, if you have the image zoomed to window size and then click the menu item to show original size, nothing happens (you have to click "zoom to window size" again to bring it back to original size).

The ctrl-shift-a toggle is fine, but then the menu entry should be renamed to "toggle between..." and the ctrl-shift-z entry can be removed (or enabled...).
Comment 1 caulier.gilles 2007-05-21 11:44:14 UTC
Daniel,

There is a conflict between Edit/Redo and View/Zoom 1:1 shortcut This is why CTRL+SHIFT+Z don't work...

Gilles
Comment 2 caulier.gilles 2007-05-21 12:03:55 UTC
SVN commit 666930 by cgilles:

digiKam from trunk : Use everywhere ALT+CTRL+0 shortcut to perform "Zoom To 1:1" action instead CTRL+SHIFT+Z witch is in conflict with Redo std shortcut defined by KDE API.
Note : ALT+CTRL+0 is also used by Photoshop 7.0
BUG: 145558

 M  +1 -1      digikam/digikamapp.cpp  
 M  +3 -2      utilities/imageeditor/editor/editorwindow.cpp  
 M  +2 -2      utilities/lighttable/lighttablewindow.cpp  


--- trunk/extragear/graphics/digikam/digikam/digikamapp.cpp #666929:666930
@@ -800,7 +800,7 @@
 
     d->zoomTo100percents = new KAction(i18n("Zoom to 1:1"), 
                                    "viewmag1",
-                                   CTRL+SHIFT+Key_Z, 
+                                   ALT+CTRL+Key_0,      // NOTE: Photoshop 7 use ALT+CTRL+0.
                                    d->view, 
                                    SLOT(slotZoomTo100Percents()),
                                    actionCollection(), 
--- trunk/extragear/graphics/digikam/utilities/imageeditor/editor/editorwindow.cpp #666929:666930
@@ -331,8 +331,9 @@
                                              actionCollection(), "editorwindow_zoomminus");
 
     d->zoomTo100percents = new KAction(i18n("Zoom to 1:1"), "viewmag1",
-                                   CTRL+SHIFT+Key_Z, this, SLOT(slotZoomTo100Percents()),
-                                   actionCollection(), "editorwindow_zoomto100percents");
+                                       ALT+CTRL+Key_0,      // NOTE: Photoshop 7 use ALT+CTRL+0. 
+                                       this, SLOT(slotZoomTo100Percents()),
+                                       actionCollection(), "editorwindow_zoomto100percents");
 
 
     d->zoomFitToWindowAction = new KToggleAction(i18n("Fit to &Window"), "view_fit_window",
--- trunk/extragear/graphics/digikam/utilities/lighttable/lighttablewindow.cpp #666929:666930
@@ -383,10 +383,10 @@
     d->zoomMinusAction->setEnabled(false);
 
     d->zoomTo100percents = new KAction(i18n("Zoom to 1:1"), "viewmag1",
-                                       CTRL+SHIFT+Key_Z, this, SLOT(slotZoomTo100Percents()),
+                                       ALT+CTRL+Key_0,      // NOTE: Photoshop 7 use ALT+CTRL+0.
+                                       this, SLOT(slotZoomTo100Percents()),
                                        actionCollection(), "lighttable_zoomto100percents");
 
-
     d->zoomFitToWindowAction = new KAction(i18n("Fit to &Window"), "view_fit_window",
                                            CTRL+SHIFT+Key_A, this, SLOT(slotFitToWindow()),
                                            actionCollection(), "lighttable_zoomfit2window");