Ubuntu Document Viewer App

Code review comment for lp:~verzegnassi-stefano/ubuntu-docviewer-app/advanced-text-editor

  1. advanced-text-editor
  2. Merge into trunk
Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

This is looking great for viewing small text files.

A few issues:-

* "Large" text files can lock up / crash the app. By "large" I mean 910kB such as https://www.gutenberg.org/cache/epub/48984/pg48984.txt

* First line number gets messed up with large files http://people.canonical.com/~alan/screenshots/device-2015-05-18-142938.png

* Edits don't get saved. If I create a small file (using vim/nano) then edit it in docviewer, and go back, the changes don't get saved to disk.

review: Needs Fixing

« Back to merge proposal