Code review comment for lp:~verzegnassi-stefano/ubuntu-docviewer-app/sgtileitem-keep-data

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

Do you mean the getter and setter for m_data?
I guess we should keep them, since we need to store the QImage returned by the RenderEngine as long as the tile is not deleted.

The Qt SceneGraph seems to repaint the node randomly several times when the tile is created and, if it does not find any QImage (because we delete it after the first paint), it just deletes the whole node, resulting in a portion of document not rendered on screen.

This is more a workaround, we probably could refine the code after the release.

« Back to merge proposal