Bug 145558 - in menu view: window size/original size: ctrl-shift-z obsolete?
Summary: in menu view: window size/original size: ctrl-shift-z obsolete?
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Usability-Keyboard (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-17 16:23 UTC by Daniel Bauer
Modified: 2017-08-08 17:03 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 0.9.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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");