Review Request: Minor GUI change to rename/overwrite dialog window
David Faure
faure at kde.org
Wed Oct 6 10:35:03 BST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5443/#review7989
-----------------------------------------------------------
Well, I would have really preferred that the astyle formatting was done in another commit; the patch is hard to read now, and svn annotate will be confusing. But nevermind, too late now. (BTW did you run kdesdk/scripts/astyle-kdelibs or something different?)
Anyway, patch seems ok now, except for the blank label. If you can test without it and it's all fine, I'll commit the patch. Thanks!
/trunk/KDE/kdelibs/kio/kio/renamedialog.cpp
<http://svn.reviewboard.kde.org/r/5443/#comment8238>
There's never a good reason for a blank label. At best a simple QWidget, but in fact in a gridlayout you can just not insert anything in the cell, and the effect should be the same.
- David
On 2010-10-06 02:43:12, Steven Sroka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5443/
> -----------------------------------------------------------
>
> (Updated 2010-10-06 02:43:12)
>
>
> Review request for kdelibs.
>
>
> Summary
> -------
>
> This is my first submission to KDE. If I am missing something, don't hesitate to tell me.
>
> This is a slight GUI change to the rename/overwrite dialog window, just to make it more user friendly.
>
>
> This addresses bug 238942.
> https://bugs.kde.org/show_bug.cgi?id=238942
>
>
> Diffs
> -----
>
> /trunk/KDE/kdelibs/kio/kio/renamedialog.h 1181964
> /trunk/KDE/kdelibs/kio/kio/renamedialog.cpp 1181964
>
> Diff: http://svn.reviewboard.kde.org/r/5443/diff
>
>
> Testing
> -------
>
> Correctly Compiled.
> "Eye ball'ed" Code.
> Ran code with all possible '_mode' as per http://api.kde.org/4.5-api/kdelibs-apidocs/kio/html/namespaceKIO.html#bac5df6792cd3426582dbfd1af706bff
> Ran many possible combinations (most if not all) -> move folder to folder, file to file, file to folder, folder to file, and paid attention to creation date.
>
> (I actually found a bug with the preview picture that is shown in a certain scenario - I will create a bug notice for it on bugs.kde.org soon)
>
>
> Screenshots
> -----------
>
> KDE Trunk - Bug
> http://svn.reviewboard.kde.org/r/5443/s/526/
> *New - File to File
> http://svn.reviewboard.kde.org/r/5443/s/527/
> KDE 4.5 - File to File
> http://svn.reviewboard.kde.org/r/5443/s/528/
> Revision 5 Screenshot
> http://svn.reviewboard.kde.org/r/5443/s/529/
> New warning
> http://svn.reviewboard.kde.org/r/5443/s/530/
>
>
> Thanks,
>
> Steven
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20101006/dd9b4a89/attachment.htm>
More information about the kde-core-devel
mailing list