D20744: [WIP] Apply new window flags only when necessary
Mariusz Glebocki
noreply at phabricator.kde.org
Mon Apr 22 19:21:57 BST 2019
mglb created this revision.
mglb added a reviewer: Konsole.
mglb added a project: Konsole.
Herald added a subscriber: konsole-devel.
mglb requested review of this revision.
REVISION SUMMARY
Applying the settings in Configure Konsole dialog causes all Konsole
windows to be hidden and shown again. This causes all information
related to window management (window state, desktop, position/display,
etc.) to be lost - windows appear on current display, desktop, at new
positions.
The hide/show action is triggered by window flags change. Only "remove
window titlebar and frame" option really changes the flags, so the
action is changed to be triggered only by this option.
Introduced changes:
- Window flags are changed change only when necessary (i.e. when "remove window titlebar and frame" option is changed).
- Windows are restored:
- at previous positions
- on the same displays
- on previous activity
- [MISSING] with the same state (minimized, maximized, etc)
- [MISSING] on the same desktop
- [MISSING] with the same stacking order
**Any ideas how to implement missing features?**
TEST PLAN
- Open a few Konsole windows, put them on different desktops, different positions, minimize, maximize, etc.
- Open Configure Konsole dialog, change any option except "remove window titlebar and frame"
- Apply
**Result before change**
Konsole windows end up on current desktop; configuration window hides
under them.
**Result after change**
Konsole windows position and state does not change.
REPOSITORY
R319 Konsole
BRANCH
wip/Apply-new-configuration-options-only-when-necessary (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D20744
AFFECTED FILES
src/MainWindow.cpp
To: mglb, #konsole
Cc: konsole-devel, #konsole, gennad, thsurrel, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20190422/435d79ec/attachment.html>
More information about the konsole-devel
mailing list