26045 2009-05-27 13:44:31 -0700 The Qt Style Sheets are not applied on webkit scrollbars 2014-02-03 03:50:33 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) PC Linux RESOLVED INVALID Qt, QtTriaged P3 Enhancement --- 1 vincent.ordy webkit-unassigned kent.hansen oldest_to_newest 123318 0 vincent.ordy 2009-05-27 13:44:31 -0700 The Qt Style Sheets set with setStyleSheet() on the QWebView are not applied on webkit scroll bars. It seems to happen because of the QStyleSheetStyleSelector::attribute() and QStyleSheetStyleSelector::nodeNames() functions used by StyleSelector::styleRulesForNode(). They both use the widget metadata to get the widget type name and properties in order to find out which rules to apply. A quick fix is to have ScrollbarThemeQt inherit from QScrollbar and use "this" as widget for drawComplexControl(). - p.drawComplexControl(QStyle::CC_ScrollBar, *opt); + p.style->drawComplexControl(QStyle::CC_ScrollBar, opt, p.painter, this); 198133 1 kent.hansen 2010-03-10 07:40:06 -0800 Please provide a testcase (in the form of a compileable Qt app, for example). 198383 2 50446 vincent.ordy 2010-03-10 15:25:58 -0800 Created attachment 50446 Testcase 198393 3 vincent.ordy 2010-03-10 15:40:11 -0800 This testcase creates 2 windows. Both have a QScrollArea which contains a QWebView. I do setStyleSheet("QScrollBar { background: blue; }") on both windows. The style of the first is QWindowsStyle, and the second's is QMotifStyle. The QWebView scrollbar does use the style set by setStyle. However, it does not use the one set by setStyleSheet("QScrollBar {...}"). Like I said before, it happens because the Qt Style Sheets system uses metadata to find out the type of an object. Since a ScrollbarThemeQt is not a QScrollbar, the style sheet for QScrollbar is not used. 198975 4 kent.hansen 2010-03-12 03:23:47 -0800 (In reply to comment #3) > This testcase creates 2 windows. Both have a QScrollArea which contains a > QWebView. I do setStyleSheet("QScrollBar { background: blue; }") on both > windows. > > The style of the first is QWindowsStyle, and the second's is QMotifStyle. The > QWebView scrollbar does use the style set by setStyle. > > However, it does not use the one set by setStyleSheet("QScrollBar {...}"). Like > I said before, it happens because the Qt Style Sheets system uses metadata to > find out the type of an object. Since a ScrollbarThemeQt is not a QScrollbar, > the style sheet for QScrollbar is not used. Thanks for the testcase. Hmm, with r33755 and Qt 4.6 trunk the QWebView scrollbars don't even respect setStyle()! It did in an older version, like you said, so this is a regression. 198976 5 kent.hansen 2010-03-12 03:24:37 -0800 (In reply to comment #4) > Hmm, with r33755 and Qt 4.6 trunk Sorry, that should be r55658. 975799 6 jturcotte 2014-02-03 03:50:33 -0800 === Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines. 50446 2010-03-10 15:25:58 -0800 2010-03-10 15:25:58 -0800 Testcase test-qt-webkit-scrollbar.tar.bz2 application/x-bzip2 1044 vincent.ordy QlpoOTFBWSZTWfDkh1oABAp/lMyQAIB6//+/f/f/gP/v//oBAQABAAhQA96pASAAA0RGkYTJpqae oaHqejU0eoDR6h6gGQ9RtGk0BoDhoZNNDTI0NMjIMjI0MgMTRk0AZMjEMcNDJpoaZGhpkZBkZGhk BiaMmgDJkYhjhoZNNDTI0NMjIMjI0MgMTRk0AZMjEMJFCARlPQ1DRgJomiZPJPSaZNoQGjQyNNNN PU9aJv19f4vu484aolKKSCQOpAC0atOuEioSlFbJkqSzZYIVgQtAjIESCGiwbt4hnwo6JZOBA7bt sS6EV0AyDGwKIVu5m2sbyW45jR7oBafHGsEVC6n/SHQiIS7kZMmp9LM4xCFsU2APeaHBfNeCqyCa 6qV8AXcchAuqQcyFGkHwv+wiuP3hIKkvM/cXrsMKacD8kSQ402HTIzCxrI09xjMcuc6F4CPmBneR 3ZRODGKJuwrwLeVnC91Mj4YeDTIqncKPlf4Xv6ZRg6pY5yltzaXR1aaadFqwaN9JmJygd4guFQVd nBPMPKwmm8g24W0w8IwgHYItEqD0wyuHlCAisYj1GX1/lwpKYRXREIiO/Csj553kUbiPooRRQKXk ouaFTVCebgkpoObFSX09sKaKY+KmhFjlGqwjPEjiRKM7dVEULFipnaKHjCMKG24whNZOTgmnOaN5 ZZZdouNaO/LVE1sxsvaKnMNbUqAR9yU3AZgwbLTAOPbvyIRhIafPjKejHRkGLeyhsig5hlSAqo/F 2PNPkUBBaozID9RQTejtd6sHFiaKN5bOTxMW6eC6Jz4Z4rwuyKzzP3hBq91ZuFe2+iL2asfa22x8 JA83BBSbcdXFUckaJZUZtfSRgmcpk/YcDCWKutsMJc2fPV8AILLJTq2fvgRNi2W6FKO9bkYeNiuP ig1gPy7HkXpvR0HsqFA2zqrCuFZ1YwJycUEvkRN/W4UL83P7CKLw5RGNUdhukBvNzGJiprLplcA6 uucigXuLhHaP/ICkfNVjZMVkA4EsDc0xpfPTVpLFz+2/P9Gmn5WlguIoOMCXFQDS0OdjHnkTELiX GSLw4uQrCw1arN7Xo5TINcpmAvkcVmxAfuIiq2FItIkw6jMc3NYcSWIwePhOdLSsqRKCcois7QJz Yc4g100UgTnXHDQ2/IQh83W8goW7etgRYhW9JxwiIXPDPWOR9lECau6888UBbOYvESORbUd4aQZx Zw4zvZdfTD0inRxtZmZWLJHBWJeqonCc5J7dSA5SevNt2bzFqZkE7sAYzNCssYrkcrmIhiFIWsXZ ICSKNvd6Pf3f9hRpGfVtRffuLmBoOgRUZ2tfcibbrPGjuRQWIYmhpJt16rGN6wTAmJpt/N0FwGGw OhVjGO1CX/i7kinChIeHJDrQ