[Panel-devel] Plasma::ProgressBar into a boxLayout

[Javier Goday]

Thanks for the anwser, everything works fine now.

so must the patch be commited?


On 11/3/07, Aaron J. Seigo <aseigo at kde.org> wrote:
> On Friday 02 November 2007, Javier Goday wrote:
> > Hello everybody.
> > I'm having problems with adding Plasma::ProgressBar's widget into a
> > boxLayout, i have done a test with Plasma::PushButtons instead of
> > ProgressBars and everything works rigth, this way:
>
> > m_layout->addItem(button);
> > m_layout->addItem(button2);
> > m_layout->addItem(bar);
>
> btw, these lines are unnecessary. items are always added to their parent's
> layouts. this is due to an interesting design decision/flaw in the current
> layout classes.
>
> > but with progressbars doesn't works :(
> > Plasma::ProgressBar *bar = new Plasma::ProgressBar(this);
> >
> > Seems that the problem is with the geometry of the progressbar,
> > if i change
> > the paintWidget method in the
> > kdebase/workspace/libs/plasma/widgets/progressbar class
> > this line
> > --> options.rect = option->rect;
> > instead of this one
> > --> options.rect = geometry().toRect();
> > everything goes right, but really don't know if this change has sense
>
> i believe this change is correct. it should only be painting the exposed
> area.
>
> --
> Aaron J. Seigo
> humru othro a kohnu se
> GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
>
> KDE core developer sponsored by Trolltech
>