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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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.