> On July 11, 2016, 4:27 p.m., Matěj Laitl wrote: > > Ship It!
Can you commit it? Sandro doesn't have commit rights AFAIK - Myriam ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128246/#review97291 ----------------------------------------------------------- On July 10, 2016, 11:13 p.m., Stefano Pettini wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128246/ > ----------------------------------------------------------- > > (Updated July 10, 2016, 11:13 p.m.) > > > Review request for Amarok. > > > 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 > >