Wrap text in copy/move confirmation dialog
ClosedPublic

Authored by gengisdave on Jul 13 2019, 4:43 PM.

Details

Summary

When you want to copy/move a file with a very long name, the confirmation dialog is opened very wide to respect the filename width.

Wrapping the text let the dialog to be drawn at the usual width.

FIXED: [ 330788 ] "Information - Krusader" dialog does not fit to screen width; text non-copyable
BUG: 330788

Test Plan

No errors in the dialog window, like buttons drawn out of the visible area.

Diff Detail

Repository
R167 Krusader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gengisdave requested review of this revision.Jul 13 2019, 4:43 PM
gengisdave created this revision.
nmel accepted this revision.Jul 20 2019, 6:44 AM
nmel added a subscriber: nmel.

It works fine but please fix coding style before pushing your changes. Thanks!

krusader/Dialogs/krdialogs.cpp
106

Please replace lbl with label according to the coding style.

This revision is now accepted and ready to land.Jul 20 2019, 6:44 AM
nmel added a comment.Jul 21 2019, 6:13 AM

If you'd like this fix to be included into v2.7.2, please merge by the end of the month.

This revision was automatically updated to reflect the committed changes.