broulik added inline comments.

INLINE COMMENTS

> meven wrote in previewjob.cpp:433
> Maybe make this static, so that apps have to do it once per app sort of like 
> we do with `Qt::AA_UseHighDpiPixmaps`, rather than by KPreviewJob.

Not a fan. You can have different dpi per screen.
Maybe instead we should have a `QWindow*` method or constructor to take dpr from

REPOSITORY
  R241 KIO

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

To: meven, dfaure, broulik, #frameworks
Cc: davidedmundson, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, 
bruns

Reply via email to