[PATCH] KHTML Fix scrollbar positions
roman.jarosz at gmail.com
Thu Feb 7 12:27:49 GMT 2008
I've sent wrong patch here's the correct one, sorry.
On Thu, 07 Feb 2008 13:21:51 +0100, Roman Jarosz <roman.jarosz at gmail.com> wrote:
> I have test it with Konqueror and I think we shouldn't clear the contentsX and contentsY
> because otherwise Konqueror won't remember contents position on reload and also painting
> won't be smooth.
> Can I commit?
> On Wed, 06 Feb 2008 23:53:16 +0100, Roman Jarosz <R.Jarosz at sh.cvut.cz> wrote:
>> I found a problem with incorrect scrollbars in KHTML.
>> When we call KHTMLPart::begin in KHTMLPart that already has some contents and scrollbars
>> aren't at position 0 then clear() is called which sets contentsX and contentsY to 0 however
>> the scrollbars remain at old positions. So the contents is painted at offset 0,0
>> but the scrollbars are at old positions. (This happens in Kopete)
>> This patch sets also scrollbars to position 0.
>> I would commit it without asking but this behavior is different from KDE3.
>> In KDE3 the position isn't cleared. So do we want to clear the scrollbar positions or should
>> we instead remove the contentsX and contentsY clearing so the position is remembered?
>> Roman Jarosz
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 554 bytes
Desc: not available
More information about the kde-core-devel