[Webkit-unassigned] [Bug 75384] [GTK] Scrollbars are drawn behind the window resize grip

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 30 08:37:11 PST 2011


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





--- Comment #6 from Carlos Garcia Campos <cgarcia at igalia.com>  2011-12-30 08:37:08 PST ---
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > (From update of attachment 120791 [details] [details] [details])
> > > View in context: https://bugs.webkit.org/attachment.cgi?id=120791&action=review
> > > 
> > > This patch should probably also add support for resizing the scrollbars. 
> > 
> > I don't get what you mean.
> > 
> > > Right now the information is just ignored. 
> > 
> > what information?
> 
> Ah, I didn't realize that there was existing support for handling the resize grip in the WebProcess! Nice.
> 
> > > We also need to consider how to handle the situation where the resize grip and the scrollbar don't intersect. 
> > 
> > how can that happen?
> 
> This can happen when the WebView widget does not intersect with the resize grip. Imagine a window that has three stacked WebViews. Only the bottom WebView intersects the resize grip. I think all you need to do is to intersect the resize grip rectangle with the WebView rectangle.

I guess that should be handled by WebProcess or WebCore code, if it's not already done, not by this patch, as I said this just sends the resize grip size to the web process.

> > I don't like it, I prefer to have two methods, because when you read that you always have to think what that 0 is.
> 
> This can be handled by adding a comment after the zero like 
> 
> ... 0 /* parameter spec */ ..

Still, having two methods doesn't hurt and they represent better what they do, the first time visibility of the resize grip hasn't changed.

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



More information about the webkit-unassigned mailing list