https://bugs.kde.org/show_bug.cgi?id=389617

Scott Harvey <sc...@spharvey.me> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/sys
                   |                            |temsettings/19712a1d22b2b40
                   |                            |55a0ff39bf57fa2fdd41388f3

--- Comment #3 from Scott Harvey <sc...@spharvey.me> ---
Git commit 19712a1d22b2b4055a0ff39bf57fa2fdd41388f3 by Scott Harvey.
Committed on 09/05/2018 at 15:00.
Pushed by sharvey into branch 'master'.

Enlarge default window size. Use (adjusted) calculation in `SizeHint` to
determine minimum window width.

Summary:
Call `setMinimumSize()` at each run to enforce a minimum size. Any enlarged
size will be saved and restored via the customary method of saving window
geometry.

Test Plan:
- Compile systemsettings with patch
- Launch system settings (systemsettings5)
- Determine that window opens at an appropriate size.
- Close System Settings and reopen; ensure size remains set

To test small or rotated screen sizes:
- Adjust display resolution to a size smaller than the standard target of
1024x700
- System settings window should launch with a margin on all sides (not
monopolize entire screen space)

Reviewers: ngraham, mart, davidedmundson, hein, #plasma

Reviewed By: ngraham, davidedmundson, hein, #plasma

Subscribers: davidedmundson, cfeck, zzag, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D12252

M  +8    -3    app/SettingsBase.cpp

https://commits.kde.org/systemsettings/19712a1d22b2b4055a0ff39bf57fa2fdd41388f3

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to