Review Request: Make pager use plasma's colorscheme

[Marco Martin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://matt.rogers.name/r/51/#review41
-----------------------------------------------------------

Ship it!


woha, now the numbers are actually readable :D
i like that you have also corrected the color scheme (foreground was... RED? bleach :))

- Marco


On 2008-02-03 11:41:12, Sebastian Kuegler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://matt.rogers.name/r/51/
> -----------------------------------------------------------
> 
> (Updated 2008-02-03 11:41:12)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Make Plasma's pager applet use kcolorscheme and the font from the taskbar. Also make Plasma's colorscheme use colors from the Oxygen palette.
> 
> With this patch, the pager should be nicely themable.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp
>   /trunk/KDE/kdebase/workspace/plasma/desktoptheme/colors
> 
> Diff: http://matt.rogers.name/r/51/diff
> 
> 
> Testing
> -------
> 
> Compiled and played around with the colors.
> 
> 
> Screenshots
> -----------
> 
> With patch
>   http://matt.rogers.name/r/51/s/3/
> Without patch
>   http://matt.rogers.name/r/51/s/4/
> 
> 
> Thanks,
> 
> Sebastian
> 
>