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/ --- (Updated Jan. 4, 2014, 8:53 a.m.) Review request for Amarok. Changes --

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

2014-01-04 Thread Nilesh Suthar
> On Jan. 3, 2014, 12:23 p.m., Matěj Laitl wrote: > > Hi, thanks for the patch. I think there could be a better approach in > > solving the bug - the one I've outlined in my review of a similar request: > > https://git.reviewboard.kde.org/r/113057/ > > > > It may turn out not possible/worth it

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

2014-01-03 Thread Matěj Laitl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114765/#review46684 --- Hi, thanks for the patch. I think there could be a better appr

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

2014-01-01 Thread Nilesh Suthar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114765/ --- Review request for Amarok. Summary (updated) - Bug 32201

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

2014-01-01 Thread Nilesh Suthar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114765/ --- (Updated Dec. 31, 2013, 5:04 p.m.) Review request for Amarok. Bugs: 322