formatting; stack view colors (Was: branches/kdevelop/3.4/languages/cpp/debugger)

[Vladimir Prus]

On Thursday 21 December 2006 03:49, Andreas Pakulat wrote:
> SVN commit 615311 by apaku:
> 
> Don't use hardcoded colors, this messes up KDE styling.
> 
> Volodya: If you think you really need 2 colors in the listview, lets discuss
> this on the list and see what color role may fit the purpose.

> --- branches/kdevelop/3.4/languages/cpp/debugger/framestackwidget.cpp #615310:615311
> @@ -38,7 +38,7 @@
>  {
>  
>  FramestackWidget::FramestackWidget(GDBController* controller,
> -                                   QWidget *parent, 
> +                                   QWidget *parent,
>                                     const char *name, WFlags f)
>          : QListView(parent, name, f),
>            viewedThread_(0),
> @@ -129,7 +129,7 @@
>  {
>      switch(e)
>      {
> -        case GDBController::program_state_changed: 
> +        case GDBController::program_state_changed:


I don't think this change corresponds to the change log? Would it be possible to
not change formatting, because next time I save this file the formatting
will be automatically fixed back and we'll have yet another huge commit.

> -void FrameStackItem::paintCell(QPainter * p, const QColorGroup & cg, 
> -                               int column, int width, int align )
> -{
> -    QColorGroup cg2(cg);
> -    if (column % 2)
> -    {
> -        cg2.setColor(QColorGroup::Base, QColor("#e4f4fe"));
> -    }
> -    QListViewItem::paintCell(p, cg2, column, width, align);
>  }

I *do* want different colors. What's the right way to grab "alternative listview color"
from QPalette or KDE style? 

- Volodya