Review Request: Pretty resize of RenameDialog according to its content

Alexey Chernov 4ernov at gmail.com
Wed Nov 16 20:55:44 GMT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103160/
-----------------------------------------------------------

(Updated Nov. 16, 2011, 8:55 p.m.)


Review request for kdelibs.


Description
-------

Several small fixes to allow RenameDialog form and its widgets resize itself according to extra information gathered for source and destination items.

Without these fixes RenameDialog in common case appears as shown on screenshot 1 (without preview for files) and 2 (without preview for files). After the fixes it looks like on screenshot 3 and 4. Screen size is respected correctly, if space doesn't allow, scrollbars appears in scroll areas as before.

What's done:
- private class contains a couple of new members to save some data between methods
- new private slot resizePanels() added to resize information scroll area after metadata for items is fetched
- preview jobs are created not at once but at the moment when metadata is fetched and the width of scroll areas is defined, it allows to render a little bigger preview to fit to size of metadata

And one question: where should I push changes instead of 'master' branch e.g. to have them merged in 4.8? They seem to be rejected for master now.


Diffs
-----

  kio/kio/renamedialog.h ddd3338ad14911808d728b359e568c1436912096 
  kio/kio/renamedialog.cpp b4cb5656d568da93e28bb56df04501600cc2364f 

Diff: http://git.reviewboard.kde.org/r/103160/diff/diff


Testing
-------

Quite thoroughly tested in case of preview and no preview, all the Qt layout issues are tested and seem to be OK.


Screenshots
-----------

RenameDialog (no file preview) without the patch
  http://git.reviewboard.kde.org/r/103160/s/338/
RenameDialog (with file preview) without the patch
  http://git.reviewboard.kde.org/r/103160/s/339/
RenameDialog (no file preview) with the patch
  http://git.reviewboard.kde.org/r/103160/s/340/
RenameDialog (with file preview) with the patch
  http://git.reviewboard.kde.org/r/103160/s/341/


Thanks,

Alexey Chernov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111116/3d9c6196/attachment.htm>


More information about the kde-core-devel mailing list