Review Request 128246: Lyrics browser switched to the same background of the every other context applet

Stefano Pettini stefano.pettini at gmail.com
Sun Aug 7 10:17:51 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128246/
-----------------------------------------------------------

(Updated Aug. 7, 2016, 10:17 a.m.)


Status
------

This change has been marked as submitted.


Review request for Amarok.


Changes
-------

Submitted with commit bdbb3a27f556c41910d3b814e5a4c73a2a93362f by Mat?j Laitl on behalf of Stefano Pettini to branch master.


Bugs: 314854
    https://bugs.kde.org/show_bug.cgi?id=314854


Repository: amarok


Description
-------

This makes the lyrics browser similar to every other context applet.
Please note that, when editing lyrics, the default textbox background
is still used.


Diffs
-----

  ChangeLog 23599f1 
  src/context/applets/lyrics/LyricsBrowser.cpp abcfe64 

Diff: https://git.reviewboard.kde.org/r/128246/diff/


Testing
-------

Tested ONLY in the following environment:
KDE Version: 4.14.16
Qt Version: 4.8.7

Tested with light and dark color scheme
Tested use case of changing KDE color scheme while Amarok is running
Tested the edit lyrics mode


Thanks,

Stefano Pettini

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20160807/928c5577/attachment.html>


More information about the Amarok-devel mailing list