Please Review my patch

2014-03-19 Thread Nilesh suthar
Hello, Can somebody review my patches and guide me if any another modification needed https://git.reviewboard.kde.org/r/114765/ https://git.reviewboard.kde.org/r/114736/ Regards, Nilesh ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde

Please Review my patch

2014-01-20 Thread Nilesh suthar
Hello, Can somebody review my patches and guide me if any another modification needed https://git.reviewboard.kde.org/r/114765/ https://git.reviewboard.kde.org/r/114736/ Regards, Nilesh ___ Amarok-devel mailing list Amarok-devel@kde.org htt

Re: Review Request 114765: Bug 322016 - Apply button is always enabled in Playlist Layout Editor dialog

2014-01-04 Thread Nilesh Suthar
/PlaylistLayoutEditDialog.cpp 99aee2a Diff: https://git.reviewboard.kde.org/r/114765/diff/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 114765: Bug 322016 - Apply button is always enabled in Playlist Layout Editor dialog

2014-01-04 Thread Nilesh Suthar
- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114765/#review46684 --- On Dec. 31, 2013, 5:04 p.m., Nilesh Suthar wrote: > > ---

Re: Review Request 114736: Bug 299431 The text in the notification area located in the lower left corner is cut when copying several tracks to the collection at the same time.

2014-01-03 Thread Nilesh Suthar
/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request 114765: Bug 322016 - Apply button is always enabled in Playlist Layout Editor dialog

2014-01-01 Thread Nilesh Suthar
99aee2a Diff: https://git.reviewboard.kde.org/r/114765/diff/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 114765: Bug 322016 - Apply button is always enabled in Playlist Layout Editor dialog

2014-01-01 Thread Nilesh Suthar
://git.reviewboard.kde.org/r/114765/diff/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request 114736: Bug 299431 Patch

2013-12-30 Thread Nilesh Suthar
Center.Changed to Vertical Top.Added Ellipsis for text overflow and tooltip to view the description Diffs - src/statusbar/ProgressBar.cpp 400390f Diff: https://git.reviewboard.kde.org/r/114736/diff/ Testing --- Thanks, Nilesh Suthar

Re: Review Request 114736: Bug 299431 The text in the notification area located in the lower left corner is cut when copying several tracks to the collection at the same time.

2013-12-30 Thread Nilesh Suthar
Diff: https://git.reviewboard.kde.org/r/114736/diff/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 114736: Bug 299431 Patch

2013-12-30 Thread Nilesh Suthar
for text overflow and tooltip to view the description Diffs - src/statusbar/ProgressBar.cpp 400390f Diff: https://git.reviewboard.kde.org/r/114736/diff/ Testing --- Thanks, Nilesh Suthar ___ Amarok-devel mailing list Amarok-devel