https://bugs.kde.org/show_bug.cgi?id=523503
--- Comment #2 from Simone Iori <[email protected]> --- Created attachment 194759 --> https://bugs.kde.org/attachment.cgi?id=194759&action=edit Comparison I have no Windows machine with a Qt build environment, so I could not try the change itself. What I could do is emulate both behaviours with a QProxyStyle on Linux and run them against a delegate that does what KPageListViewDelegate does: draw PE_PanelItemViewItem through the style, then the text in HighlightedText. Screenshot attached, six cases. "Qt 6.9" fills with backgroundBrush if one is set and does nothing otherwise, and paints the selection in CE_ItemViewItem. "patched" paints the selection in PE_PanelItemViewItem as the change does. The change fixes the case with no modification to KPageView at all (bottom left of the screenshot), which is the result that matters here. With the KWidgetsAddons workaround applied on top of it, the outcome is identical, no double painting: the new code takes the highlight brush when the item is selected and only falls back to backgroundBrush when it is not, so the brush the delegate sets is simply ignored in that case. On unpatched Qt the workaround does make the selection visible again, but as a square fill over the whole item rect rather than the rounded selection, since that is what fillRect() with the given brush produces. So it reads as a stopgap for the Qt versions that do not have the change, not as the way it should look. Please treat this as what it is: an emulation of the two code paths, not a test of the real style. Someone with Windows 11 and a Qt built from the change should still confirm it. -- You are receiving this mail because: You are watching all bug changes.
