Bug 104116 - [Qt][Mac] QWebView disappears when the system tries to hide the scrollbars.
Summary: [Qt][Mac] QWebView disappears when the system tries to hide the scrollbars.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zeno Albisser
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-05 06:33 PST by Zeno Albisser
Modified: 2012-12-11 03:22 PST (History)
3 users (show)

See Also:

Attachments
Patch (2.85 KB, patch)
2012-12-05 06:43 PST, Zeno Albisser 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Zeno Albisser 2012-12-05 06:33:46 PST 
[Qt][Mac] QWebView disappears when the system tries to hide the scrollbars.
Comment 1 Zeno Albisser 2012-12-05 06:43:03 PST 
Created attachment 177738 [details]
Patch
Comment 2 Simon Hausmann 2012-12-05 07:00:01 PST 
Comment on attachment 177738 [details]
Patch

Ohh, since the widget pointer in the style option was the webview, the animation of fading out the scrollbar actually operated on the webview? Hehehe

LGTM.
Comment 3 Zeno Albisser 2012-12-05 07:32:12 PST 
Comment on attachment 177738 [details]
Patch

Clearing flags on attachment: 177738

Committed r136685: <http://trac.webkit.org/changeset/136685>
Comment 4 Zeno Albisser 2012-12-05 07:32:17 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Eike Ziller 2012-12-11 01:24:07 PST 
This is actually not really fixed, e.g. Qt Creator uses a custom style "ManhattenStyle" which is a QProxyStyle to the system style (which on Mac is QMacStyle), so it still happens there.
Comment 6 Pierre Rossi 2012-12-11 03:20:00 PST 
(In reply to comment #5)
> This is actually not really fixed, e.g. Qt Creator uses a custom style "ManhattenStyle" which is a QProxyStyle to the system style (which on Mac is QMacStyle), so it still happens there.

Seems like we would need an equivalent of inherits for style proxying in order for the condition used here to be more thorough.
Comment 7 Zeno Albisser 2012-12-11 03:22:20 PST 
(In reply to comment #6)
> Seems like we would need an equivalent of inherits for style proxying in order for the condition used here to be more thorough.

I think that could be achieved by querying QProxyStyle::baseStyle(), no?
But it feels like yet another workaround.