> On Nov. 17, 2011, 9:20 a.m., Aaron J. Seigo wrote: > > looks very nice and thorough. > > > > as for branch: it's a bug fix for sure and should go into the KDE/4.7 > > branch. that gets merged into master on a semi-regular basis, so don't > > worry about master (it'll happen for you). it will also get merged into > > frameworks. cheers ... > > Alexey Chernov wrote: > Thanks, Aaron! > Will push it to KDE/4.7 in the evening.
Thanks, please close bug 265657 when you commit. - Christoph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103160/#review8254 ----------------------------------------------------------- On Nov. 16, 2011, 8:55 p.m., Alexey Chernov wrote: > > ----------------------------------------------------------- > 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 > >