ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  I really wish that KCMshell didn't add its own scrollview, so we didn't end 
up with issues like this. The KCM itself should be responsible for its own 
layouting needs.
  
  Then again I also wish we didn't have KCMshell at all and that opening a 
specific KCM just opened system settings/kinfocenter. It would eliminate this 
entirely class of kcmshell sizing bugs immediately.
  
  Oh well, LGTM.

REPOSITORY
  R102 KInfoCenter

BRANCH
  Plasma/5.18

REVISION DETAIL
  https://phabricator.kde.org/D28678

To: sitter, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to