Code review comment for lp:~mrqtros/ubuntu-docviewer-app/reboot-qsg-impress-support

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

Overall it looks good to me! Excellent work! :)

Anyway there are a few things that require a fix (mostly some bad-looking piece of code I wrote, or deprecated stuff imported from PDF code).
Since I've already fixed six times the 'go to position' dialog in LibreOffice viewer (bzr conflicts), I'd prefer to push those fixes in this branch, and do the work just once.

It would be really appreciated if you could include the following changes in your branch:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-impress-support-mp-review/+merge/274068

« Back to merge proposal