KFileWidget: Set minimum size for zoom slider
ClosedPublic

Authored by rkflx on Apr 29 2018, 6:07 AM.

Details

Summary

When choosing a very narrow width for the file dialog, the zoom slider
for the icon size would get squished so that only the handle would be
visible and the slider could not be used anymore to change zoom levels
in any way.

This can be avoided by setting a minimum size. 40px are smaller than the
default size, but still allow to operate the slider comfortably.

The minimal width of the dialog is unchanged, as before the toolbar will
show the elision button once elements start overflowing.

Ref T8552

FIXED-IN: 5.46

Test Plan

kdialog --getopenfilename, change window size and observe zoom slider.

Diff Detail

Repository
R241 KIO
Branch
slider-min-size (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
rkflx created this revision.Apr 29 2018, 6:07 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 29 2018, 6:07 AM
rkflx requested review of this revision.Apr 29 2018, 6:07 AM
ngraham accepted this revision.Apr 29 2018, 10:42 PM
ngraham added a subscriber: ngraham.

Perfecto.

This revision is now accepted and ready to land.Apr 29 2018, 10:42 PM
This revision was automatically updated to reflect the committed changes.