D13766: fixed picture scaling issues in epub generator

[Gilbert Assaf]

gassaf added a comment.


  I did some more tests and epub_get_data() always returned the correct size.
  
  Original image file from ebook:
  F5980094: A978-1-4302-6653-2_CoverFigure_HTML.jpg <https://phabricator.kde.org/F5980094>
  Screenshot from Okular, build with
  
    /*if(img.height() > maxHeight)
        img = img.scaledToHeight(maxHeight);
    if(img.width() > maxWidth)
        img = img.scaledToWidth(maxWidth);*/
  
  F5980120: Without_scaling.png <https://phabricator.kde.org/F5980120>
  
  So I would argue Okular does load all images with the correct size. HOWEVER if you use Okulars zoom functionality the image gets blurry again:
  F5980159: without_scaling_blurry.png <https://phabricator.kde.org/F5980159>
  
  I don't know for sure what causes this behavior. Maybe this is a QTextDocument issue? I will try to look into this.

REPOSITORY
  R223 Okular

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

To: gassaf, #okular
Cc: ngraham, aacid, okular-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20180630/b5bd46c9/attachment-0001.html>