D27533: [WIP] Add MarkInterfaceV2, to s/QPixmap/QIcon/g for symbols of marks

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Thu Feb 20 23:44:32 GMT 2020


kossebau added inline comments.

INLINE COMMENTS

> kateviewhelpers.cpp:1963
>                              const int s = qMin(m_iconAreaWidth * devicePixelRatioF(), h * devicePixelRatioF()) - 2;
> -                            px_mark = px_mark.scaled(s, s, Qt::KeepAspectRatio, Qt::SmoothTransformation);
> +                            QPixmap px_mark = markIcon.pixmap(s);
> +                            px_mark.setDevicePixelRatio(devicePixelRatioF());

Possibly `QIcon::paint()` might be also working here as wanted? Needs someone with HiDPI to check if all things behave as wanted. The old code with all the `devicePixelRatioF()` made me change not too much here.

REPOSITORY
  R39 KTextEditor

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

To: kossebau, #kate, #kdevelop
Cc: kwrite-devel, kde-frameworks-devel, rrosch, LeGast00n, cblack, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200220/a36c427c/attachment.html>


More information about the Kde-frameworks-devel mailing list