Review Request: Makes Notes applet remember the Notes color change

[Manuel Tortosa]

> On Dec. 7, 2011, 8:15 p.m., Manuel Tortosa wrote:
> > Ship It!

Tested, works perect.


- Manuel


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103353/#review8786
-----------------------------------------------------------


On Dec. 7, 2011, 7:58 p.m., Anne-Marie Mahfouf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103353/
> -----------------------------------------------------------
> 
> (Updated Dec. 7, 2011, 7:58 p.m.)
> 
> 
> Review request for Plasma and Aaron J. Seigo.
> 
> 
> Description
> -------
> 
> The Notes applet does not apply the Notes color change, nor by right click or via the Settings dialog. The new color is not written in configuration. This was because m_topWidget->color() always returns yellow-notes as a color. 
> This patch introduces a new variable to store the color without the -notes and match it against the colors action menu.
> 
> Manuel, can you test please?
> 
> 
> This addresses bugs 288286 and 288286.
>     http://bugs.kde.org/show_bug.cgi?id=288286
>     http://bugs.kde.org/show_bug.cgi?id=288286
> 
> 
> Diffs
> -----
> 
>   applets/notes/notes.h fde13cc 
>   applets/notes/notes.cpp 10a8753 
> 
> Diff: http://git.reviewboard.kde.org/r/103353/diff/diff
> 
> 
> Testing
> -------
> 
> Tested and works as expected now, both on Desktop and dashboard.
> 
> 
> Thanks,
> 
> Anne-Marie Mahfouf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111207/c87b1349/attachment-0001.html>