Differential D21986

Change initial size of the config dialog
ClosedPublic
Actions

Authored by nicolasfella on Jun 22 2019, 10:38 AM.

Details

Reviewers
apol
ngraham
Group Reviewers
VDG
Commits
R495:e6e0be782237: Change initial size of the config dialog
Summary

The dialog is larger than it needs to be. Make it smaller

Diff Detail

Repository
R495 Purpose Library
Branch
size
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13128
Build 13146: arc lint + arc unit
nicolasfella created this revision.Jun 22 2019, 10:38 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptJun 22 2019, 10:38 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Jun 22 2019, 10:38 AM
nicolasfella edited the summary of this revision. (Show Details)Jun 22 2019, 10:40 AM
aacid added a subscriber: aacid.Jun 22 2019, 11:04 AM

How hard is to make the size be related to the contents instead of a magic number?

ngraham added a comment.Jun 22 2019, 11:22 AM
In D21986#484013, @aacid wrote:

How hard is to make the size be related to the contents instead of a magic number?

+1, this would probably involve basing the size on the StackView's child items and bubbling that up to the top, rather than setting a hardcoded size at the top and making all the child items do anchors.fill: parent

If that's ridiculously difficult due to other bugs though, this is a reasonable improvement as-is.

ngraham accepted this revision.Jun 23 2019, 9:17 PM

I looked into what it would take to make the size adjust to its contents and I don't think it's worth the effort here due to the way this dialog is constructed. Porting to use a real QtQuick dialog would solve a lot of problems, but until we do that, this change is fine.

This revision is now accepted and ready to land.Jun 23 2019, 9:17 PM
nicolasfella retitled this revision from RFC: Change initial size of the config dialog to Change initial size of the config dialog.Jun 24 2019, 4:09 PM
Closed by commit R495:e6e0be782237: Change initial size of the config dialog (authored by nicolasfella). · Explain WhyJun 24 2019, 4:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/widgets/JobDialog.qml (4 lines)

Diff 60310

View Options

src/widgets/JobDialog.qml

Loading...