apol added a comment.

  +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.

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

Reply via email to