[Kde-pim] Review Request: Use colors from current color scheme

[Christoph Feck]

> On 2010-09-21 23:36:45, Allen Winter wrote:
> > please commit this.
> > I hope the previous commit I made for review5211 doesn't conflict in some way.

It does :)

(and 5211 uses different color roles than discussed here, but I won't argue again)


- Christoph


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/2131/#review7719
-----------------------------------------------------------


On 2009-11-29 02:48:35, Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/2131/
> -----------------------------------------------------------
> 
> (Updated 2009-11-29 02:48:35)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Unless configured otherwise, Akregator uses hardcoded red and blue colors for new and unread articles. This patch changes the default colors to use the recommended colors from the current color scheme.
> 
> The only problem is that with current default color theme (Oxygen), the "new/active" text is a somewhat bright pink, which does not contrast well on the white background. I have reported bug 216599 against Oxygen to inform them about this issue.
> 
> 
> This addresses bug 207860.
>     https://bugs.kde.org/show_bug.cgi?id=207860
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdepim/akregator/src/articlelistview.cpp 1047245 
> 
> Diff: http://svn.reviewboard.kde.org/r/2131/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Christoph
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/