Version: (using KDE KDE 3.5.2) ... at least this is where I (and my family members, on whom I did a little statistic research) would expect it. I checked other KDE apps and most of them have it under "Settings" like it is now in digikam. Is this a written KDE standard? Then it might stay that way. Otherwise I think "View" is way more intuitive (user thinks "i want to view my album fullscreen" rather then "I have to change the setting to "fullscreen mode"). No long discussions here though - feel free to close this bug if you think the current location is "better" in some way.
SVN commit 626745 by lure: Digikam Editor and ShowFoto menu improvements: - move View menu after Edit menu - move Full Screen Mode from Settings to View menu BUG: 140320 CCBUG: 127617 M +18 -18 showfoto/showfotoui.rc M +16 -16 utilities/imageeditor/editor/digikamimagewindowui.rc --- trunk/extragear/graphics/digikam/showfoto/showfotoui.rc #626744:626745 @@ -1,5 +1,5 @@ <!DOCTYPE kpartgui SYSTEM "kpartgui.dtd"> -<gui version="9" name="showfoto" > +<gui version="10" name="showfoto" > <MenuBar> @@ -31,6 +31,23 @@ <Action name="editorwindow_redo" /> </Menu> + <Menu name="View" ><text>&View</text> + <Action name="editorwindow_fullscreen" /> + <Separator/> + <Action name="editorwindow_zoomplus" /> + <Action name="editorwindow_zoomminus" /> + <Action name="editorwindow_zoomfit" /> + <Separator/> + <Action name="shofoto_showthumbs" /> + <Action name="editorwindow_histogram" /> + <Separator/> + <Action name="editorwindow_slideshow" /> + <Separator/> + <Action name="editorwindow_underexposure" /> + <Action name="editorwindow_overexposure" /> + <Action name="editorwindow_cmview" /> + </Menu> + <Menu name="Image" ><text>&Image</text> <Separator/> </Menu> @@ -46,21 +63,6 @@ <Action name="editorwindow_resize" /> </Menu> - <Menu name="View" ><text>&View</text> - <Action name="editorwindow_zoomplus" /> - <Action name="editorwindow_zoomminus" /> - <Action name="editorwindow_zoomfit" /> - <Separator/> - <Action name="shofoto_showthumbs" /> - <Action name="editorwindow_histogram" /> - <Separator/> - <Action name="editorwindow_slideshow" /> - <Separator/> - <Action name="editorwindow_underexposure" /> - <Action name="editorwindow_overexposure" /> - <Action name="editorwindow_cmview" /> - </Menu> - <Menu name="help" ><text>&Help</text> <Action name="editorwindow_imagepluginshelp" /> <Separator/> @@ -73,8 +75,6 @@ <Action name="options_show_toolbar" /> <Merge name="StandardToolBarMenuHandler" /> <Separator/> - <Action name="editorwindow_fullscreen" /> - <Separator/> <Action name="options_configure_keybinding" /> <Action name="options_configure_toolbars" /> <Action name="options_configure" /> --- trunk/extragear/graphics/digikam/utilities/imageeditor/editor/digikamimagewindowui.rc #626744:626745 @@ -1,5 +1,5 @@ <!DOCTYPE kpartgui SYSTEM "kpartgui.dtd"> -<gui version="7" name="digikamimagewindow" > +<gui version="8" name="digikamimagewindow" > <MenuBar> @@ -28,20 +28,9 @@ <Action name="editorwindow_redo" /> </Menu> - <Menu name="Image" ><text>&Image</text> - </Menu> - - <Menu name="Fix" ><text>Fi&x</text> - </Menu> - - <Menu name="Transform" ><text>&Transform</text> - <Action name="editorwindow_rotate" /> - <Action name="editorwindow_flip" /> - <Action name="editorwindow_crop" /> - <Action name="editorwindow_resize" /> - </Menu> - <Menu name="View" ><text>&View</text> + <Action name="editorwindow_fullscreen" /> + <Separator/> <Action name="editorwindow_zoomplus" /> <Action name="editorwindow_zoomminus" /> <Action name="editorwindow_zoomfit" /> @@ -55,6 +44,19 @@ <Action name="editorwindow_cmview" /> </Menu> + <Menu name="Image" ><text>&Image</text> + </Menu> + + <Menu name="Fix" ><text>Fi&x</text> + </Menu> + + <Menu name="Transform" ><text>&Transform</text> + <Action name="editorwindow_rotate" /> + <Action name="editorwindow_flip" /> + <Action name="editorwindow_crop" /> + <Action name="editorwindow_resize" /> + </Menu> + <Menu name="help" ><text>&Help</text> <Action name="editorwindow_imagepluginshelp" /> <Separator/> @@ -67,8 +69,6 @@ <Action name="options_show_toolbar"/> <Merge name="StandardToolBarMenuHandler" /> <Separator/> - <Action name="editorwindow_fullscreen" /> - <Separator/> <Action name="options_configure_keybinding"/> <Action name="options_configure_toolbars"/> <Action name="options_configure" />
Similar change has to be done also for Digikam and option should be renamed to "Full Screen Mode" to match Editor/ShowFoto and Konqueror.
SVN commit 626869 by lure: Full screen action improvement for Digikam: - move action from Settings menu to View menu - use KStdAction::fullScreen() on KDE 3.2 or newer (improved toggle support) - use KToggleAction on older KDE This makes it consistent with Editor, ShowFoto and other KDE applications. BUG: 127617 M +6 -1 digikamapp.cpp M +3 -3 digikamui.rc --- trunk/extragear/graphics/digikam/digikam/digikamapp.cpp #626868:626869 @@ -736,7 +736,11 @@ "album_thumbSizeDecrease"); mThumbSizeMinusAction->setWhatsThis(i18n("This option allows you to decrease the Album thumbnails size.")); - mFullScreenAction = new KAction(i18n("Toggle Full Screen"), +#if KDE_IS_VERSION(3,2,0) + mFullScreenAction = KStdAction::fullScreen(this, SLOT(slotToggleFullScreen()), + actionCollection(), this, "full_screen"); +#else + mFullScreenAction = new KToggleAction(i18n("Full Screen Mode"), "window_fullscreen", CTRL+SHIFT+Key_F, this, @@ -744,6 +748,7 @@ actionCollection(), "full_screen"); mFullScreenAction->setWhatsThis(i18n("This option allows you to toggle the main windows in full screen mode.")); +#endif mQuitAction = KStdAction::quit(this, SLOT(slot_exit()), --- trunk/extragear/graphics/digikam/digikam/digikamui.rc #626868:626869 @@ -1,5 +1,5 @@ <!DOCTYPE kpartgui SYSTEM "kpartgui.dtd"> -<kpartgui version="16" name="digikam" > +<kpartgui version="17" name="digikam" > <MenuBar> @@ -59,6 +59,8 @@ <Menu name="View" > <text>&View</text> + <Action name="full_screen" /> + <Separator/> <Action name="album_thumbSizeIncrease" /> <Action name="album_thumbSizeDecrease" /> <Separator /> @@ -88,8 +90,6 @@ <Action name="options_show_toolbar"/> <Merge name="StandardToolBarMenuHandler" /> <Separator/> - <Action name="full_screen" /> - <Separator/> <Action name="theme_menu" /> <Action name="options_configure_keybinding"/> <Action name="options_configure_toolbars"/>