D8590: KPageListView: Update width on font change

Henrik Fehlauer noreply at phabricator.kde.org
Mon Nov 6 16:17:17 UTC 2017


rkflx accepted this revision.
rkflx added a comment.


  Since I was added as a reviewer:
  
  - Tested with `KWordQuiz`: No app restart necessary anymore to adapt width to changed font.
  - Code looks simple enough and even passes the event to the base class.
  
  Perfect patch, thanks :)
  
  (Minor comment: You could have added some instructions to the test plan, saving me some time.)

REPOSITORY
  R236 KWidgetsAddons

BRANCH
  KPageListView-updateWidth

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

To: volkov, cfeck, wojnilowicz, dfaure, rkflx, broulik
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171106/198d59b5/attachment.html>


More information about the Kde-frameworks-devel mailing list