[mini-patch] make scrolling stop when moving view via API
Garand Germain
germain at ebooksfrance.org
Sat Aug 29 00:16:54 BST 2009
Le dimanche 23 août 2009, Martin Koller a écrit :
> Hi,
>
> The bug https://bugs.kde.org/show_bug.cgi?id=184273
> mentions that KHTMLView will not stop auto scrolling when one presses key
> up/down.
> This is due to the fact the KHTMLView does not stop it's internal timer
> when one calls its scrollBy() Method, which is what kmail does.
>
> I suggest the following patch.
> Is this ok for commit ?
>
>
> Index: khtmlview.cpp
> ===================================================================
> --- khtmlview.cpp (Revision 1013649)
> +++ khtmlview.cpp (Arbeitskopie)
> @@ -746,6 +746,8 @@
>
> void KHTMLView::scrollBy(int x, int y)
> {
> + if (d->scrollTimerId)
> + d->newScrollTimer(this, 0);
> horizontalScrollBar()->setValue( horizontalScrollBar()->value()+x );
> verticalScrollBar()->setValue( verticalScrollBar()->value()+y );
> }
Hi Martin,
the patch looks fine : please commit.
Greetings,
Germain
More information about the kfm-devel
mailing list