D12252: Enlarge window size: wide enough to show both sidebarsUse (adjusted) calculation in `SizeHint` to determine minimum window width to display both sidebars. Manually apply size with call to `resize()`

Scott Harvey noreply at phabricator.kde.org
Mon Apr 16 17:39:45 UTC 2018


sharvey marked an inline comment as done.
sharvey added inline comments.

INLINE COMMENTS

> ngraham wrote in SettingsBase.cpp:159
> Don't remove this, or else it becomes possible to resize the window to absurdly, unusably small sizes. The value could be changed, but we shouldn't remove this entirely.

Moved to line 130; keep together with initial window size setting. I went with an arbitrary but common 800x600 for the minimum size.

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D12252

To: sharvey, ngraham, mart
Cc: zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180416/2ac353de/attachment.html>


More information about the Plasma-devel mailing list