Review Request 108905: Move UI code of PlaylistQueueEditor from .ui to .cpp file
Bartosz Szreder
szreder at mimuw.edu.pl
Mon Feb 11 10:39:05 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108905/
-----------------------------------------------------------
Review request for Amarok.
Description
-------
This is the first patch (of two) resolving bug #263563
I need to create a specialized QListWidget implementation. Thus it seems easier to move the rather limited part of UI into the .cpp file and cleanly add another class with the desired modifications than to dig into .ui files with nonstandard classes.
This patch shouldn't introduce any changes in behavior yet. It's only purpose is to prepare ground for the next patch.
This addresses bug 263563.
https://bugs.kde.org/show_bug.cgi?id=263563
Diffs
-----
src/playlist/PlaylistQueueEditor.h 40b8cdf
src/playlist/PlaylistQueueEditor.cpp f647e37
src/playlist/PlaylistQueueEditor.ui a05cafd
src/CMakeLists.txt 043dc64
Diff: http://git.reviewboard.kde.org/r/108905/diff/
Testing
-------
Tested on two different instalations, no regressions noticed.
Thanks,
Bartosz Szreder
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130211/97cd56ec/attachment.html>
More information about the Amarok-devel
mailing list