<table><tr><td style="">ndavis created this revision.<br />ndavis added reviewers: VDG, Breeze.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ndavis requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26572">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>QQC2/Kirigami checkboxes and radio buttons can turn invisible when rendered over a selected item because their background isn't rendered and they don't have any hacks to detect when they're being rendered over a selected list item.</p>

<p>While the problem isn't technically a Breeze QStyle problem and a hack could be made for QQC2/Kirigami, I don't think there's any great style benefit to not rendering a background for the checkbox. I suppose there is a performance benefit to not rendering a checkbox background except for when the background is different from normal. In my testing with GammaRay's paint analyzer, the checkbox background has a cost less than 5%. The radiobutton background has a cost of 15-20% (maybe it can be improved?).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>BRANCH</strong><div><div>checkbox-radiobutton-background (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26572">https://phabricator.kde.org/D26572</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>.clangd/index/breeze.h.4FF33B56EAE2D7BA.idx<br />
.clangd/index/breeze.h.7B10C7466ADC2AA0.idx<br />
.clangd/index/breezeaddeventfilter.cpp.D0E662BB28BE244D.idx<br />
.clangd/index/breezeaddeventfilter.h.9BABBE41CDB0D66F.idx<br />
.clangd/index/breezeanimation.cpp.275B156AFAB98527.idx<br />
.clangd/index/breezeanimation.h.693D1D99A9910A4B.idx<br />
.clangd/index/breezeanimationdata.cpp.4317A521E2DE2E2F.idx<br />
.clangd/index/breezeanimationdata.h.BC8F15C60A3034AB.idx<br />
.clangd/index/breezeanimations.cpp.C3E8FECE039B9FC5.idx<br />
.clangd/index/breezeanimations.h.5192379C11257929.idx<br />
.clangd/index/breezebaseengine.cpp.E83D566AFC278B4E.idx<br />
.clangd/index/breezebaseengine.h.505F11F7C6A87C62.idx<br />
.clangd/index/breezeblurhelper.cpp.AEC559F830AF17CC.idx<br />
.clangd/index/breezeblurhelper.h.C94B6F166C77187E.idx<br />
.clangd/index/breezeboxshadowrenderer.cpp.786229CDEA661D34.idx<br />
.clangd/index/breezeboxshadowrenderer.h.BC29F28B1740A3FC.idx<br />
.clangd/index/breezebusyindicatordata.cpp.1D6336C113AA80C6.idx<br />
.clangd/index/breezebusyindicatordata.h.DCD0DB12FADFCD79.idx<br />
.clangd/index/breezebusyindicatorengine.cpp.18A08078CD46CED1.idx<br />
.clangd/index/breezebusyindicatorengine.h.F624C75A30174FFC.idx<br />
.clangd/index/breezebutton.cpp.90886A3EBA068FEF.idx<br />
.clangd/index/breezebutton.h.530A068407DE2943.idx<br />
.clangd/index/breezeconfigwidget.cpp.50BB1F341C21C81E.idx<br />
.clangd/index/breezeconfigwidget.h.6F970B64E9C4907B.idx<br />
.clangd/index/breezedatamap.h.1C2EC78CB6B70291.idx<br />
.clangd/index/breezedecoration.cpp.D8E265288DDAC43D.idx<br />
.clangd/index/breezedecoration.h.A5778114F9E22002.idx<br />
.clangd/index/breezedetectwidget.cpp.EA9486E12A28BD5F.idx<br />
.clangd/index/breezedetectwidget.h.E2628B233F59D17F.idx<br />
.clangd/index/breezedialdata.cpp.35842EB928B480E0.idx<br />
.clangd/index/breezedialdata.h.AAEAD6CE0E86858C.idx<br />
.clangd/index/breezedialengine.cpp.4AD170FD3E70705B.idx<br />
.clangd/index/breezedialengine.h.3B19DD1F09077A0B.idx<br />
.clangd/index/breezeenabledata.cpp.D4493FE613990E6E.idx<br />
.clangd/index/breezeenabledata.h.D82CACA43D4D2295.idx<br />
.clangd/index/breezeexceptiondialog.cpp.6F9302774CF68C34.idx<br />
.clangd/index/breezeexceptiondialog.h.CD37D1206C6E2A0F.idx<br />
.clangd/index/breezeexceptionlist.cpp.2F7B4D10845DF367.idx<br />
.clangd/index/breezeexceptionlist.h.94D46BB9AE84A9CA.idx<br />
.clangd/index/breezeexceptionlistwidget.cpp.A9EE923843C03A49.idx<br />
.clangd/index/breezeexceptionlistwidget.h.AE38C771A5426EEE.idx<br />
.clangd/index/breezeexceptionmodel.cpp.BAA536AF418E6C15.idx<br />
.clangd/index/breezeexceptionmodel.h.91F5C64C32658FD8.idx<br />
.clangd/index/breezeframeshadow.cpp.B6BDDD99CEE326A3.idx<br />
.clangd/index/breezeframeshadow.h.F5B9EF8E98711482.idx<br />
.clangd/index/breezegenericdata.cpp.885FE94E6FF9E89E.idx<br />
.clangd/index/breezegenericdata.h.897C9E80FE921A2C.idx<br />
.clangd/index/breezeheaderviewdata.cpp.11F02F991FA17AC5.idx<br />
.clangd/index/breezeheaderviewdata.h.0BEE45C48B1B955E.idx<br />
.clangd/index/breezeheaderviewengine.cpp.FB9805E2FAB8BC8D.idx<br />
.clangd/index/breezeheaderviewengine.h.C88453BF3AF1A052.idx<br />
.clangd/index/breezehelper.cpp.3C462F86BFF5E5A0.idx<br />
.clangd/index/breezehelper.h.E38F1B4A9A26E28E.idx<br />
.clangd/index/breezeitemmodel.cpp.84413A312586CA51.idx<br />
.clangd/index/breezeitemmodel.h.7D0F974D26694300.idx<br />
.clangd/index/breezelistmodel.h.8B7D38E8B751D201.idx<br />
.clangd/index/breezemdiwindowshadow.cpp.280BE053A40C011A.idx<br />
.clangd/index/breezemdiwindowshadow.h.C2175FF1D49039F1.idx<br />
.clangd/index/breezemnemonics.cpp.EAEBB3411BAF9A5D.idx<br />
.clangd/index/breezemnemonics.h.355D80624A3E15D4.idx<br />
.clangd/index/breezepropertynames.cpp.E2389F07C3E61543.idx<br />
.clangd/index/breezepropertynames.h.A31791E8DAD35DEF.idx<br />
.clangd/index/breezescrollbardata.cpp.6AB2BC04084A6E9C.idx<br />
.clangd/index/breezescrollbardata.h.757AF4C9E13D9096.idx<br />
.clangd/index/breezescrollbarengine.cpp.6E3A3059FC5041C1.idx<br />
.clangd/index/breezescrollbarengine.h.7B1BFDEF270906E5.idx<br />
.clangd/index/breezesettingsprovider.cpp.7269A737CBA450D6.idx<br />
.clangd/index/breezesettingsprovider.h.276FEB6BFF2527D8.idx<br />
.clangd/index/breezeshadowhelper.cpp.5683ECBD133DFE7C.idx<br />
.clangd/index/breezeshadowhelper.h.4BD4EC4A995B0689.idx<br />
.clangd/index/breezesizegrip.cpp.FC68DA193A1431EB.idx<br />
.clangd/index/breezesizegrip.h.956D54695E10038A.idx<br />
.clangd/index/breezespinboxdata.cpp.00A4C84ED065EA6A.idx<br />
.clangd/index/breezespinboxdata.h.F4C6DE5EE901F3DB.idx<br />
.clangd/index/breezespinboxengine.cpp.D9D49602BA35761E.idx<br />
.clangd/index/breezespinboxengine.h.72C3875239E51B9A.idx<br />
.clangd/index/breezesplitterproxy.cpp.3165B6819AC84749.idx<br />
.clangd/index/breezesplitterproxy.h.1E3367F82813CB5A.idx<br />
.clangd/index/breezestackedwidgetdata.cpp.12C53B58511B5628.idx<br />
.clangd/index/breezestackedwidgetdata.h.8B76B85FFF9B8593.idx<br />
.clangd/index/breezestackedwidgetengine.cpp.83AAC16F892DD0BB.idx<br />
.clangd/index/breezestackedwidgetengine.h.6DDAA5C80463F4DD.idx<br />
.clangd/index/breezestyle.cpp.D55EE06D4FFD1EC8.idx<br />
.clangd/index/breezestyle.h.725BEA00CFEF0B06.idx<br />
.clangd/index/breezestyleconfig.cpp.9A03CEE2A496D0B9.idx<br />
.clangd/index/breezestyleconfig.h.2F74CDA02B255AB0.idx<br />
.clangd/index/breezestyleconfigmodule.cpp.6EB58B07C6351162.idx<br />
.clangd/index/breezestyleconfigmodule.h.09D73A12B9059F20.idx<br />
.clangd/index/breezestyleplugin.cpp.F26F14A37514E4D1.idx<br />
.clangd/index/breezestyleplugin.h.90F06B839763B950.idx<br />
.clangd/index/breezetabbardata.cpp.1A039D935B690C2F.idx<br />
.clangd/index/breezetabbardata.h.CB1AC7670CB920FE.idx<br />
.clangd/index/breezetabbarengine.cpp.FA6E6AE84CC7AB01.idx<br />
.clangd/index/breezetabbarengine.h.367912CE325769F9.idx<br />
.clangd/index/breezetileset.cpp.511AB81907809F73.idx<br />
.clangd/index/breezetileset.h.153A483EF79538E9.idx<br />
.clangd/index/breezetoolboxengine.cpp.05B6AAA34CE11A94.idx<br />
.clangd/index/breezetoolboxengine.h.11F527E5493A38FD.idx<br />
.clangd/index/breezetransitiondata.cpp.616EABF0DD500321.idx<br />
.clangd/index/breezetransitiondata.h.CC7F1C5C59E8F8BD.idx<br />
.clangd/index/breezetransitionwidget.cpp.3E97507618EBAE91.idx<br />
.clangd/index/breezetransitionwidget.h.B51A3E983B874D00.idx<br />
.clangd/index/breezewidgetexplorer.cpp.3029CC6756AD3A6D.idx<br />
.clangd/index/breezewidgetexplorer.h.9CE77D10E9F2D708.idx<br />
.clangd/index/breezewidgetstatedata.cpp.F2D4E8996DE27CE9.idx<br />
.clangd/index/breezewidgetstatedata.h.E47B172022AB1247.idx<br />
.clangd/index/breezewidgetstateengine.cpp.E0F6DE02CEDE08BE.idx<br />
.clangd/index/breezewidgetstateengine.h.3B3043E5B7F2735A.idx<br />
.clangd/index/breezewindowmanager.cpp.F8A96D8829FEE6B1.idx<br />
.clangd/index/breezewindowmanager.h.626AFAAEA98B6F6C.idx<br />
.clangd/index/main.cpp.1050EE6306A54994.idx<br />
.clangd/index/main.cpp.5813F9DF676A4A90.idx<br />
kstyle/breezestyle.cpp</div></div></div><br /><div><strong>To: </strong>ndavis, VDG, Breeze<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>