[Bug 214107] New: show/hide toolbar forgotten when switching fullscreen mode

classic Classic list List threaded Threaded
26 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] New: show/hide toolbar forgotten when switching fullscreen mode

Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

           Summary: show/hide toolbar forgotten  when switching fullscreen
                    mode
           Product: digikam
           Version: unspecified
          Platform: Compiled Sources
        OS/Version: unspecified
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:           svn 1046770 (using KDE 4.3.2)
Installed from:    Compiled From Sources

Hi,

If you use fullscreen mode, then you hide the toolbar using right click context
menu, if you switch to normal mode and go back to fullscreen mode the toolbar
is shown again, I think it should not be the case.

I think two different settings should be saved :
- show/hide toolbar in fullscreen mode
- show/hide toolbar in normal mode

Julien

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
https://bugs.kde.org/show_bug.cgi?id=214107


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
            Version|unspecified                 |1.0.0-beta6




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Marcel Wiesweg
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107


Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #1 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-12-20 15:20:35 ---
The option to hide the toolbar in full-screen mode can be found in the image
editor tab of the settings. You're right that it's not discoverable, it should
better be changed in-place and saved for each window.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107





--- Comment #2 from Gilles Caulier <caulier gilles gmail com>  2009-12-20 18:13:36 ---
Marcel,

It's the common place in menu used by all KDE applications...

Gilles

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Bugzilla from Julien@narboux.fr
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107





--- Comment #3 from Julien Narboux <Julien narboux fr>  2009-12-20 20:24:38 ---
Marcel, Gilles,

I think the setting is discoverable, but what is strange is that if you hide
the toolbar using the context menu on the toolbar itself it does not change
Digikam's setting.
Moreover if you hide the toolbar in normal mode using context menu. If you go
fullscreen and then back to normal mode, the toolbar is shown again, I think it
shouldn't.

Julien

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|1.0.0-beta6                 |1.0.0




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #4 from Gilles Caulier <caulier gilles gmail com>  2011-12-12 20:35:46 ---
*** Bug 288487 has been marked as a duplicate of this bug. ***

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #5 from Gilles Caulier <caulier gilles gmail com>  2011-12-18 10:22:06 ---
*** Bug 192424 has been marked as a duplicate of this bug. ***

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

julien.t43+kde
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #6 from [hidden email] ---
Bug 236234 (https://bugs.kde.org/show_bug.cgi?id=236234) is related but for
sidebars.
Also this problem affects both status and menubar.
Another option which could help would be keyboard shortcuts to make each bar
appear/disappear. don't believe it's currently possible.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #7 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #8 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #9 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #10 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Veaceslav Munteanu
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #11 from Veaceslav Munteanu <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #12 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #13 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, bug 236234, bug 258180, bug 293676

M  +88   -40   libs/widgets/common/dxmlguiwindow.cpp

http://commits.kde.org/digikam/d6ddab1db393a973f90fbe628ae49b0b91bfe3b7

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #14 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #15 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, bug 236234, bug 258180, bug 293676

M  +34   -4    libs/widgets/common/dxmlguiwindow.cpp

http://commits.kde.org/digikam/99601b3df7c7837069b90e1744c89b2070a7a832

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #16 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 214107] show/hide toolbar forgotten when switching fullscreen mode

Gilles Caulier-4
In reply to this post by Bugzilla from Julien@narboux.fr
https://bugs.kde.org/show_bug.cgi?id=214107

--- Comment #17 from Gilles Caulier <[hidden email]> ---
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 220739, bug 289262, 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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
12