https://bugs.kde.org/show_bug.cgi?id=431945

--- Comment #5 from Tiar <tamtamy.tym...@gmail.com> ---
Looks like it's inevitable: performance is more important... you're welcome to
make a MR, but please:
1) add a comment explaining why preview.png is used instead of mergedimage.png
(so no one else makes the same mistake again),
2) keep other changes; for example the devicePixelRatioF() stuff (it keeps it
looking at least somewhat nice on 4k screens). (I'm afraid the pixel art won't
look good there anyway, so it might be a good idea to remove that part that
scales them up using FastTransformation; it will be blurred terribly anyway,
unfortunately... SmoothTransformation at least will have only one kind of
artifacts).
3) It would be good if you checked if after your changes the pages in the page
viewer are still rendered nicely with high resolution.

Thank you :)

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to