[Webkit-unassigned] [Bug 25646] [GTK] onscroll event is never sent

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun May 10 10:59:22 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=25646


xan.lopez at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |xan.lopez at gmail.com




------- Comment #3 from xan.lopez at gmail.com  2009-05-10 10:59 PDT -------
(In reply to comment #2)
> And using this we can ditch the createNativeScrollbar, you mean?
> 
> > -        /*
> > -         * disable the scrollbars (if we have any) as the GtkAdjustment over
> > -         */
> 
> I like the patch in general, it's a good cleanup, but this comment has really
> blown off my mind while I was reading through this code last week, and I
> believe you could perhaps make it clearer at this point why we need to disable
> the scrollbars here:

That's because when you set the adjustments you are telling the View that
you'll handle the scrolling yourself (for example, through a
GtkScrolledWindow), so you want to disable WebKit's scrollbars. I'm not sure
why Holger is removing the comment though...


> > +        m_horizontalAdjustment->setSteps(visibleWidth() / 10.0, visibleWidth() * 0.9, 1);
> > +        m_horizontalAdjustment->setProportion(visibleWidth(), contentsWidth());
> > +        m_horizontalAdjustment->setCurrentPos(scroll.width());
> 
> Xan has done some work recently on the scrolling code to use the WebCore
> constants instead of magic numbers, which I think we should use here as well;
> See r43390.

/me nods (although the WebCore constants are more 'magic numbers' than what we
were using :D)

I'll try to look at this tomorrow!


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list