davidre added a comment.
In D29154#656585 <https://phabricator.kde.org/D29154#656585>, @apol wrote: > +1 > > I did the same thing for kirigami @ D29100 <https://phabricator.kde.org/D29100> and plasma-framework @ D29102 <https://phabricator.kde.org/D29102> yesterday after looking at Qt's code and assessing it was the right thing to do. > I am not familiar with Breeze's code but the patch is sound. > > I do have all of the dpis so if you tell me how I'll test it. > > This should probably fix the BUG 418869. I guess how you tested your patches? Looking at icons on all the dpis? REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D29154 To: davidre, apol, broulik, ndavis, #breeze, davidedmundson, cblack Cc: cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
