[Patch] scrollbars

Germain Garand germain at ebooksfrance.org
Tue Aug 3 14:03:27 BST 2004


Le Mardi 03 Août 2004 09:55, Stephan Kulow a écrit :
> Am Dienstag 03 August 2004 05:49 schrieb Germain Garand:
> > Likewise, relevant testcases from #61730 look fine (#59,#65,#68,#72) ;
> > #69613 does not regress.
>
> I didn't have a look at the regression test cases yet, but I browsed
> through the test cases on bugs.kde.org and found this one to kill konqueror
> http://bugs.kde.org/attachment.cgi?id=3697

mmh right, the patch address the case of one scrollbar triggering the other,
but if both are needed, we still rely on QScrollView to do the Right Thing ;(

I'm investigating if anything can be done from our side.

>
> The worst I found while browsing were horizontal scrollbars not going away
> for width=100%, e.g.
> http://bugs.kde.org/attachment.cgi?id=4073
>

I have a minimal version of this testcase. This is actually "expected result" 
for our renderer. Ie: that's another bug (setting marginwidth=0 in 
combination with 100% width makes the container slightly bigger than 100%).
So the scrollbar doesn't go away because there actually is something to show.

Any other dubious instances of that behaviour ?

> Note, that I only tested with your patch on a fresh CVS, so I can't say if
> these are regressions or were there before.
>
> Greetings, Stephan





More information about the kfm-devel mailing list