Bug 414682 - "Download new global themes" window does not fit to laptop screen (1366x768)
Summary: "Download new global themes" window does not fit to laptop screen (1366x768)
Status: RESOLVED FIXED
Alias: None
Product: frameworks-knewstuff
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords:
: 416398 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-11-30 11:25 UTC by Patrick Silva
Modified: 2020-02-07 14:38 UTC (History)
6 users (show)

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


Attachments
screenshot (235.51 KB, image/png)
2019-11-30 11:25 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2019-11-30 11:25:25 UTC
Created attachment 124202 [details]
screenshot

STEPS TO REPRODUCE
1. open system settings > global theme
2. click on "Download new global themes" button
3. 

OBSERVED RESULT
as we can see in the attached screenhot, bottom of the window is out of the screen.

EXPECTED RESULT
the window fits to the screen

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.17.80
KDE Frameworks Version: 5.65.0
Qt Version: 5.13.2
Comment 1 Antonio Rojas 2020-01-19 10:02:07 UTC
*** Bug 416398 has been marked as a duplicate of this bug. ***
Comment 2 Dan Leinir Turthra Jensen 2020-02-07 10:33:07 UTC
Just uploaded a patch to attempt to alleviate this issue:
https://phabricator.kde.org/D27204
Comment 3 Dan Leinir Turthra Jensen 2020-02-07 13:53:16 UTC
Git commit fd5b37818e6aa72fc149d9aec58511ca83e8a9d4 by Dan Leinir Turthra Jensen.
Committed on 07/02/2020 at 13:53.
Pushed by leinir into branch 'master'.

Fix the KNSQuick Dialog's close title and basic layout

Summary:
This resolves several issues found in the KNSQuick Dialog component.

Specifically it puts a background in the dialog (as a rectangle),
since the QtQuick Dialogs base dialog lacks such a thing by default,
and offers no background property to rectify it.

It further changes the Close button from a custom one to using a
default button provided by the various underlying systems. This
is more fault proof than the previous incarnation, but requires
that the escape button press is handled separately.

Finally it sets a sensible default size (previously the dialog
was made large by the layout, but as a side effect of the layouting
now being done without a Layout (which we could, but which would make
the code enormously convoluted for little gain), we can now set a
set of implicit width and height on the content item, which results
in both a sensible default size and a dialogue that will resize down
the way one would expect it to).

This could have been done as separate patches, but they're too
intertwined for that to feel in any way reasonable to do...
Related: bug 415606, bug 416354

Test Plan:
The dialog with its new default size looks as follows:
{F8089818}

Reviewers: #frameworks, ngraham

Reviewed By: ngraham

Subscribers: kde-frameworks-devel

Tags: #frameworks

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

M  +19   -13   src/qtquick/qml/Dialog.qml

https://commits.kde.org/knewstuff/fd5b37818e6aa72fc149d9aec58511ca83e8a9d4