Bug 397001 - The "maximise" button is missing in decoration of the geolocation editor window [patch]
Summary: The "maximise" button is missing in decoration of the geolocation editor wind...
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Generic-GeolocationEdit (show other bugs)
Version: 6.0.0
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-31 04:23 UTC by Milan Knížek
Modified: 2018-08-13 00:23 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0
Sentry Crash Report:


Attachments
windowFlags.patch (776 bytes, patch)
2018-08-01 20:01 UTC, Maik Qualmann
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Milan Knížek 2018-07-31 04:23:40 UTC
Hi,

It would be usefull to have the editor opened in a standard window - with minimise, maximise and restore buttons, rather than the close button only.

Currently, the window never opens maximised and it has to be manually resized ba dragging the window borders.

Arch Linux, GNOME, AppImage 6.0.0 beta.
Comment 1 Maik Qualmann 2018-07-31 06:46:52 UTC
Here under a Plasma desktop are all buttons available. GNOME believes that a dialog window does not require buttons to maximize or minimize. As far as I know, you need to install a tweak program to re-enable these windows buttons.

Maik
Comment 3 Milan Knížek 2018-07-31 16:13:18 UTC
That guy had a problem with Cinnamon, which may not be the same as standard GNOME.

I have gnome-tweaks installed, disabled "attach modal dialogs" (so that the dialog window moves separately from the parent window), enabled Maximise and Minimise buttons for the title bar already.

That said, Tweaks do not seem to offer a solution for me.

You are right, that GNOME treats Geolocation Editor as a dialog window (in Alt+Tab switcher, I can see both digikam and editor as a single icon). Right-clicking on Editor's title bar shows both Maximise and Minimise greyed out (disabled), double click does not maximise the window. Note that the window has a title bar and the close button.

I searched a bit more, but no resolution yet. This one suggests that Mac OS might deal with modal windows similarly like GNOME does:
https://musescore.org/en/node/56931

P.S. Re-opening the bug if you do not mind.
Comment 4 Maik Qualmann 2018-08-01 20:01:06 UTC
Created attachment 114257 [details]
windowFlags.patch

I've installed a GNOME3 desktop for testing. Although openSUSE creates a very good basic configuration, I have to say it is not my desktop. But it does not matter. This patch removes the Qt::Dialog flag and replaced it with the Qt::Window flag. Now it is possible to maximize and minimize under GNOME. But we have to test whether it has negative effects in other environments.

Maik
Comment 5 Maik Qualmann 2018-08-12 10:56:40 UTC
Git commit 62499189cbe39308bff2315a98cfd21f3a7397f5 by Maik Qualmann.
Committed on 12/08/2018 at 10:43.
Pushed by mqualmann into branch 'master'.

replace Qt::Dialog window flag with Qt::Window to enable min/max button under GNOME
currently tested for problems under GNOME3, KF5 and Window7/10
BUGS: 397001
FIXED-IN: 6.0.0

https://commits.kde.org/digikam/62499189cbe39308bff2315a98cfd21f3a7397f5
Comment 6 Maik Qualmann 2018-08-13 00:23:52 UTC
Git commit 62499189cbe39308bff2315a98cfd21f3a7397f5 by Maik Qualmann.
Committed on 12/08/2018 at 10:43.
Pushed by mqualmann into branch 'master'.

replace Qt::Dialog window flag with Qt::Window to enable min/max button under GNOME
currently tested for problems under GNOME3, KF5 and Window7/10
FIXED-IN: 6.0.0

M  +2    -2    NEWS
M  +4    -0    core/utilities/advancedrename/advancedrenamedialog.cpp
M  +4    -0    core/utilities/geolocation/editor/dialog/geolocationedit.cpp
M  +4    -0    core/utilities/maintenance/maintenancedlg.cpp
M  +3    -0    core/utilities/mediaserver/dmediaserverdlg.cpp
M  +3    -0    core/utilities/metadataedit/dialog/metadataedit.cpp
M  +3    -0    core/utilities/setup/setup.cpp

https://commits.kde.org/digikam/62499189cbe39308bff2315a98cfd21f3a7397f5