| Summary: | Hitting ESC in "fullscreen picture preview" messes up thumbbar-state in "thumbnail view" | ||
|---|---|---|---|
| Product: | [Applications] digikam | Reporter: | Peter Albrecht <px79> |
| Component: | Usability-FullScreen | Assignee: | Digikam Developers <digikam-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | caulier.gilles, julien.t43+kde, smit.meh |
| Priority: | NOR | ||
| Version: | 3.0.0 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/digikam/a8e10da1a0fb247ff6c361de4f5069ec77e4115c | Version Fixed In: | 3.2.0 |
| Attachments: |
screenshot with normal window
screenshot in full screen mode Screenshot of digikam after step 6 (in the "Steps to Reproduce" list) |
||
*** Bug 278346 has been marked as a duplicate of this bug. *** See also wish at Bug #289264 I'm not sure that "Bug 278346 - status bar is not preserved in full screen mode"" is a duplicate. The problem is still valid in 2.5 (will test when upgraded to 2.6) and affects also menubar in full screen. See also Bug Bug 214107. status bar still not preserved in full screen mode in 2.6 still valid in 2.8.0 (kde 4.8.5, lubuntu 12.04 ppa). worse, even if the status bar is not displayed, progress manager still pop up with some kind of screen artifacts. I tested recently behavior of menubar and toolbar on 2.8.0 and they are preserved between full screen/non full mode so what's the difference between status bar ? Still valid in digikam 2.9.0 with KDE 4.8.5. Hi Peter, julien.t43+kde, Could you please provide a couple of screenshots? Smit Created attachment 74762 [details]
screenshot with normal window
Created attachment 74763 [details]
screenshot in full screen mode
Here screenshots from normal to fullscreen mode, toolbar disabled and status preserved (if enabled, kept visible), menubar enabled and disappeared (but if disabled in normal mode, it will still be invisible in fullscreen mode and when back in normal mode), statusbar disabled and disappear (to make appear, need either the toolbar, either to have attributed a shortcut)
Created attachment 74766 [details]
Screenshot of digikam after step 6 (in the "Steps to Reproduce" list)
This is, what digikam looks like, after replaying the "steps to reproduce" list. In the upper rectangle you see thumbnails in the thumbbar and in the lower rectangle, you see thumbnails in "main client area" (how do you call this?).
Displaying two thumbnail lists of the same fotos does not make very much sense. ;)
Don't know about the steps to reproduce, have not written them and not using the thumbnail bar (no space for it ...) My bug (Bug 278346 - status bar is not preserved in full screen mode) was more about status bar than thumbnail bar. I don't even find where to enable thumbnail bar be it with menu or with toolbar. seems only in config file $ grep -i thumb ~/.kde/share/config/digikamrc |grep -i bar Show Thumbbar=false FullScreenHideThumbBar=true Show Thumbbar=false To enable the thumbbar (in digikam 2.9.0): In the main menu: 1. View -> Preview Image (F3) 2. View -> Show Thumbbar (CTRL + T) (10th entry in the View menu) Confirmed, easy to reproduce with the steps provided in the initial post. The code which restores the thumbbar is probably fooled by the switch in full screen mode back to the thumbnail view. Normal behavior is restored when switch to preview view and back. Git commit 73502cc716f1940375d7d805e299d8f93fa58adc by Gilles Caulier. Committed on 30/04/2013 at 06:02. Pushed by cgilles into branch 'master'. Add new FullScreenMngr class dedicated manage full-screen action to digiKam KXMLGuiWindow instances. This class factorize a lots of code when window is turned on/off to full-screen mode. This fix problem about visibility restoring with status-bar, menu-bar, and thumb-bar. First managed digiKam main window is Image Editor. TODO : - manage full-screen mode of main windows AlbumGUI, BQM, ImportUI, and LT, through FullScreenMngr. - manage visibility of side-bars. - provide a common settings widget to control visibility management in Configuration dialog. - patch Configuration dialog to provide a fine control of visibility management in full-screen mode for each digiKam main window. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +1 -0 CMakeLists.txt A +231 -0 libs/widgets/common/fullscreenmngr.cpp [License: GPL (v2+)] A +89 -0 libs/widgets/common/fullscreenmngr.h [License: GPL (v2+)] M +19 -109 utilities/imageeditor/editor/editorwindow.cpp M +0 -5 utilities/imageeditor/editor/editorwindow.h M +5 -13 utilities/imageeditor/editor/editorwindow_p.h M +12 -8 utilities/setup/setupeditor.cpp http://commits.kde.org/digikam/73502cc716f1940375d7d805e299d8f93fa58adc Git commit cde52bd90737bd8fe2b7dfdb94eba09578549239 by Gilles Caulier. Committed on 01/05/2013 at 14:19. Pushed by cgilles into branch 'master'. Port fullscreen mode from Light Table to FullScreenMngr. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +23 -87 utilities/lighttable/lighttablewindow.cpp M +1 -3 utilities/lighttable/lighttablewindow.h M +3 -8 utilities/lighttable/lighttablewindow_p.h M +13 -10 utilities/setup/setuplighttable.cpp M +1 -1 utilities/setup/setuplighttable.h http://commits.kde.org/digikam/cde52bd90737bd8fe2b7dfdb94eba09578549239 Git commit 4b034a235c07758d97b1fe763791abe1bc3c81db by Gilles Caulier. Committed on 01/05/2013 at 15:10. Pushed by cgilles into branch 'master'. Port fullscreen mode from Batch Queue Manager to FullScreenMngr. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +3 -0 libs/widgets/common/fullscreenmngr.h M +23 -79 utilities/queuemanager/main/queuemgrwindow.cpp M +1 -1 utilities/queuemanager/main/queuemgrwindow.h M +4 -8 utilities/queuemanager/main/queuemgrwindow_p.h http://commits.kde.org/digikam/4b034a235c07758d97b1fe763791abe1bc3c81db Git commit b8a7f5ceddc5c112906e94eaedeb42cbb0917638 by Gilles Caulier. Committed on 02/05/2013 at 16:10. Pushed by cgilles into branch 'master'. As under OSX, if toolbar is hidden during fullscreen mode, we display fullscreen switch button on top-right corner of active sceen when mouse is moved on this corner by end user. Mouse move envent is handle by an event filter plug over managed window. FullScreenMngr is now a QObject class based. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +63 -1 libs/widgets/common/fullscreenmngr.cpp M +9 -1 libs/widgets/common/fullscreenmngr.h M +0 -1 utilities/imageeditor/editor/editorwindow.cpp M +0 -1 utilities/lighttable/lighttablewindow.cpp M +0 -1 utilities/queuemanager/main/queuemgrwindow.cpp http://commits.kde.org/digikam/b8a7f5ceddc5c112906e94eaedeb42cbb0917638 Git commit 115ff76386c0835ed4f647b30eb6f80eac387d5f by Veaceslav Munteanu, on behalf of Gilles Caulier. Committed on 02/05/2013 at 16:10. Pushed by munteanu into branch 'picassafacetag'. As under OSX, if toolbar is hidden during fullscreen mode, we display fullscreen switch button on top-right corner of active sceen when mouse is moved on this corner by end user. Mouse move envent is handle by an event filter plug over managed window. FullScreenMngr is now a QObject class based. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +63 -1 libs/widgets/common/fullscreenmngr.cpp M +9 -1 libs/widgets/common/fullscreenmngr.h M +0 -1 utilities/imageeditor/editor/editorwindow.cpp M +0 -1 utilities/lighttable/lighttablewindow.cpp M +0 -1 utilities/queuemanager/main/queuemgrwindow.cpp http://commits.kde.org/digikam/115ff76386c0835ed4f647b30eb6f80eac387d5f Git commit 8dcfa0b102f4b4d53a5095b8daae0efc1582a4da by Gilles Caulier. Committed on 03/05/2013 at 11:33. Pushed by cgilles into branch 'master'. For a better factoring and better management of all common features from digiKam KXmlGuiWindow based main windows a new class named DXmlGuiWindow have been created. All digiKam main windows will be based on. Currently IE, BQM, and LT are done. AlbumGUI and ImportUI still TODO. This class replace FullScreenMngr as well. FullScreen mode are no managed through DXmlGuiWindow. Later more factoring will be done in this class, as all Help digiKam dialogs (DB stats, Components info, etc...) Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +2 -1 CMakeLists.txt M +7 -3 digikam/utils/albumsettings.cpp M +3 -0 digikam/utils/albumsettings.h D +0 -404 libs/widgets/common/fullscreenmngr.cpp D +0 -140 libs/widgets/common/fullscreenmngr.h A +104 -0 libs/widgets/common/fullscreensettings.cpp [License: GPL (v2+)] A +61 -0 libs/widgets/common/fullscreensettings.h [License: GPL (v2+)] M +19 -43 utilities/imageeditor/editor/editorwindow.cpp M +3 -8 utilities/imageeditor/editor/editorwindow.h M +0 -3 utilities/imageeditor/editor/editorwindow_p.h M +8 -27 utilities/lighttable/lighttablewindow.cpp M +3 -4 utilities/lighttable/lighttablewindow.h M +0 -3 utilities/lighttable/lighttablewindow_p.h M +6 -24 utilities/queuemanager/main/queuemgrwindow.cpp M +2 -4 utilities/queuemanager/main/queuemgrwindow.h M +0 -4 utilities/queuemanager/main/queuemgrwindow_p.h M +2 -1 utilities/setup/setupeditor.cpp M +2 -1 utilities/setup/setuplighttable.cpp http://commits.kde.org/digikam/8dcfa0b102f4b4d53a5095b8daae0efc1582a4da Git commit d6ddab1db393a973f90fbe628ae49b0b91bfe3b7 by Gilles Caulier. Committed on 03/05/2013 at 22:38. Pushed by cgilles into branch 'master'. Check toolbars visibility before FullScreen mode and restore it after. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +88 -40 libs/widgets/common/dxmlguiwindow.cpp http://commits.kde.org/digikam/d6ddab1db393a973f90fbe628ae49b0b91bfe3b7 Git commit 5511e68fcb1054c292beff81f0f2e5e4635f550d by Gilles Caulier. Committed on 03/05/2013 at 22:54. Pushed by cgilles into branch 'master'. Check thumbbar visibility before FullScreen mode and restore it after. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +20 -10 libs/widgets/common/dxmlguiwindow.cpp M +4 -0 libs/widgets/common/dxmlguiwindow.h M +5 -0 utilities/imageeditor/editor/editorwindow.cpp M +1 -0 utilities/imageeditor/editor/editorwindow.h http://commits.kde.org/digikam/5511e68fcb1054c292beff81f0f2e5e4635f550d Git commit 99601b3df7c7837069b90e1744c89b2070a7a832 by Gilles Caulier. Committed on 03/05/2013 at 23:03. Pushed by cgilles into branch 'master'. Check menubar and statusbar visibilities before FullScreen mode and restore it after. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +34 -4 libs/widgets/common/dxmlguiwindow.cpp http://commits.kde.org/digikam/99601b3df7c7837069b90e1744c89b2070a7a832 Git commit a1bb7a21fc367966beba3d6a1af81c9b201b6544 by Gilles Caulier. Committed on 04/05/2013 at 19:15. Pushed by cgilles into branch 'master'. Manage FullScreen mode from AlbumGUI through DXmlGuiWindow Polish API Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +17 -83 digikam/main/digikamapp.cpp M +6 -6 digikam/main/digikamapp.h M +7 -0 libs/widgets/common/dxmlguiwindow.cpp M +9 -4 libs/widgets/common/dxmlguiwindow.h M +7 -21 utilities/imageeditor/editor/editorwindow.cpp M +1 -2 utilities/imageeditor/editor/editorwindow.h http://commits.kde.org/digikam/a1bb7a21fc367966beba3d6a1af81c9b201b6544 Git commit 392c928cec8b5092cd383b328002a5fa7652fa16 by Gilles Caulier. Committed on 04/05/2013 at 20:39. Pushed by cgilles into branch 'master'. Manage FullScreen mode from ImportUI through DXmlGuiWindow Polish API Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +7 -0 digikam/main/digikamapp.cpp M +2 -5 digikam/main/digikamapp_p.h M +19 -101 utilities/importui/main/importui.cpp M +6 -7 utilities/importui/main/importui.h M +0 -8 utilities/importui/main/importui_p.h http://commits.kde.org/digikam/392c928cec8b5092cd383b328002a5fa7652fa16 Git commit 9d2bb7da9fdd805cc7c23d3d9ba3ecc5d247a48d by Gilles Caulier. Committed on 06/05/2013 at 09:28. Pushed by cgilles into branch 'master'. add new Full-screen option to hide side-bars Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +1 -1 digikam/main/digikamapp.cpp M +1 -1 digikam/main/digikamapp.h M +18 -9 libs/widgets/common/dxmlguiwindow.cpp M +8 -6 libs/widgets/common/dxmlguiwindow.h M +17 -5 libs/widgets/common/fullscreensettings.cpp M +1 -1 utilities/imageeditor/editor/editorwindow.cpp M +1 -1 utilities/imageeditor/editor/editorwindow.h M +1 -1 utilities/importui/main/importui.cpp M +1 -1 utilities/importui/main/importui.h M +2 -2 utilities/lighttable/lighttablewindow.cpp M +1 -1 utilities/lighttable/lighttablewindow.h M +0 -20 utilities/queuemanager/main/queuemgrwindow.cpp M +0 -2 utilities/queuemanager/main/queuemgrwindow.h M +1 -1 utilities/setup/setuplighttable.cpp http://commits.kde.org/digikam/9d2bb7da9fdd805cc7c23d3d9ba3ecc5d247a48d Git commit 52cdc362797c6eb92d7768d1cad7b67e4833055f by Gilles Caulier. Committed on 06/05/2013 at 09:59. Pushed by cgilles into branch 'master'. Image Editor and Light Table windows now manage side-bars visibility in Full-Screen mode. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +1 -1 libs/widgets/common/dxmlguiwindow.cpp M +7 -5 libs/widgets/common/dxmlguiwindow.h M +2 -1 libs/widgets/common/fullscreensettings.cpp M +1 -1 utilities/imageeditor/editor/editorwindow.cpp M +1 -1 utilities/lighttable/lighttablewindow.cpp M +1 -1 utilities/setup/setupeditor.cpp M +1 -1 utilities/setup/setuplighttable.cpp http://commits.kde.org/digikam/52cdc362797c6eb92d7768d1cad7b67e4833055f Git commit 8cda9359d08eeff8712203be7271aee29c841231 by Gilles Caulier. Committed on 06/05/2013 at 10:13. Pushed by cgilles into branch 'master'. Add fullscreen mode options to AlbumGUI : toolbars, thumbbar, and sidebars are managed. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 M +1 -1 digikam/main/digikamapp.cpp M +1 -0 libs/widgets/common/dxmlguiwindow.h M +40 -24 utilities/setup/setupalbumview.cpp http://commits.kde.org/digikam/8cda9359d08eeff8712203be7271aee29c841231 Git commit a8e10da1a0fb247ff6c361de4f5069ec77e4115c by Gilles Caulier. Committed on 06/05/2013 at 10:28. Pushed by cgilles into branch 'master'. Add fullscreen mode options to ImportUI : toolbars, thumbbar, and sidebars are managed. Related: bug 214107, bug 220739, bug 236234, bug 258180, bug 293676 FIXED-IN: 3.2.0 M +2 -1 libs/widgets/common/dxmlguiwindow.h M +1 -1 utilities/importui/main/importui.cpp M +19 -4 utilities/setup/setupcamera.cpp M +1 -1 utilities/setup/setupcamera.h http://commits.kde.org/digikam/a8e10da1a0fb247ff6c361de4f5069ec77e4115c |
Version: 2.4.0 (using KDE 4.7.3) OS: Linux Switching through a few picture view modes, you can and up in "thumbnail view with a thumbbar enabled". Reproducible: Always Steps to Reproduce: 1. start digikam 2. select album with picture -> thumbnail-view (no thumbnail-bar on top) 3. doubleclick/ENTER -> switch from "thumbnail view" to "image preview" (thumbnail-bar visible on top) 4. CTRL + SHIFT + F -> full screen view, but no thumbnail-bar visible on top 5. Hit ESC -> switches from "full-screen preview mode" to "full-screen thumbnail view" (no thumbbar on top) 6. leave "full-screen mode" with CTRL + SHIFT + F Actual Results: Non-full-screen thumbnail with __with thumbbar enabled__. This is funny, but not very useful. It only fills up unnecessary space on the screen. Also the toolbar-button "Show Thumbbar" (if added manually) is disabled, so you can't turn the thumbbar off. Same applies to Shortcut: CTRL + T Expected Results: In thumbnail-view, the thumbbar should be disabled. You can fix this state by switching to "preview mode" (doubleclick / ENTER) and back (ESC). So it is no extreme workflow blocker, but it is annoying and confusing.