14:05 <popey> #startmeeting Docviewer meeting
14:05 <meetingology> Meeting started Thu May 21 14:05:40 2015 UTC.  The chair is popey. Information about MeetBot at http://wiki.ubuntu.com/meetingology.
14:05 <meetingology> 
14:05 <meetingology> Available commands: action commands idea info link nick
14:05 <popey> how we doing?
14:06 <sverzegnassi> didn't do much this week, but I have some further progress with zoom \o/
14:07 <JMulholland> nice
14:07 <sverzegnassi> i've fixed the biggest issue there was (pages were destroyed while zooming, so that all the coordinates were wrong)
14:08 <sverzegnassi> I also had a look at the performance issue with the text editor component, and may require some discussion
14:08 <popey> ok. What's up?
14:09 <sverzegnassi> ref. https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/advanced-text-editor/+merge/257943
14:09 <sverzegnassi> there are two things that make the loading of the documents slow:
14:10 <sverzegnassi> 1) line numbering, but that can be solved
14:10 <sverzegnassi> 2) the binding between the file reader and the TextArea. The fix for it may be a little tricky
14:12 <sverzegnassi> So here's my question: do we want just to make the UI working with no lag (without taking care of the time the component need to load the file)?
14:14 <popey> hmmm
14:16 <sverzegnassi> ok. there was also some design question that I'd like to discuss...
14:17 <sverzegnassi> I asked to Kevin for the application icon, but I got no answer yet
14:17 <sverzegnassi> (BTW new default apps icons are awesome!)
14:18 <popey> They are!
14:18 <popey> if Kevin has no time for ie, Maybe we can ask Matthieu if he has some time to create one.
14:18 <popey> (given he made all the others)
14:19 <sverzegnassi> yeah, it's not that big priority for now. could be a nice solution
14:19 <popey> So in answer to your question, yes I think the UI shouldn't lag for displaying plain text files.
14:20 <sverzegnassi> ok, I have already thought a solution for it, but it could make the things a bit (not much) slower. I'll have a look net week, after my exam (it's on Monday)
14:20 <popey> ok.
14:20 <popey> Btw I only noticed the little SD card icon on documents this week.
14:20 <popey> Really like that!
14:20 <sverzegnassi> Can we top-approve it then?
14:21 <popey> yes
14:21 * popey clicks
14:21 <popey> done
14:22 <popey> will upload a little later.
14:22 <popey> thanks sverzegnassi
14:22 <sverzegnassi> np, thanks
14:22 <sverzegnassi> Second question about design is: https://bugs.launchpad.net/ubuntu-docviewer-app/+bug/1453403
14:22 <sverzegnassi> i.e. the hint for the header behaviour
14:23 <popey> I see olga has a design task.
14:23 <JMulholland> yea, the ‘sliding up’ behaviour is the norm all over our apps/platform
14:24 <popey> ok.
14:24 <JMulholland> so we should probably align with that if possible
14:24 <popey> Thanks JMulholland
14:24 <sverzegnassi> FWIW the current behaviour has been decided some months ago with the design team
14:25 <JMulholland> FYI: The header component for the SDK is currently under review and Olga is working on that on the SDK side, which is most likely why this was assigned to her instead of me
14:25 <JMulholland> The work is toward convergence/other form factors though, so as far as I know this ‘slide up’ behaviour isn’t planned to change
14:26 <sverzegnassi> huh, ok. I've missed the update in the bug status, sorry
14:28 <sverzegnassi> popey: as I replied in the TextEditor MP review, may it be worth to ask to the ux team suggestions on what we should do this plain text files?
14:28 <sverzegnassi> (allow editing of the file, show an hint for the read-only mode, etc...)
14:29 <popey> personally I think the file should be read-only
14:30 <sverzegnassi> ok, what about showing an hint?
14:30 <sverzegnassi> JMulholland: we're talking about https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/advanced-text-editor/+merge/257943
14:31 <sverzegnassi> I suggested to show something similar to: https://imgur.com/aEY4a8T
14:32 <popey> So if we assume this is a document _viewer_ and will never be an "editor" then we shouldn't have to tell people the file is read only
14:32 <JMulholland> if it’s read only it would be nice to let the user know, agreed
14:32 <popey> we don't tell people a pdf is read only
14:32 <JMulholland> i’ll see if we have any existing applicable UI patterns
14:32 <popey> people should assume from the name "viewer" that it's a "Read only view" on files.
14:32 <JMulholland> good point
14:33 <popey> if we choose to add an edit option, then sure, we should allow the user to toggle mode and make that clear
14:33 <popey> but I'm not convinced this needs to be able to edit files.
14:33 <popey> (we ship vi and nano on the phone) :D
14:33 <sverzegnassi> Yep, I'd prefer to have a separate app (or an built-in editor in the file manager)
14:34 <sverzegnassi> I'm talking about the look of the whole thing, which could me suggest it works like an advanced editor
14:34 <popey> ok.
14:36 <sverzegnassi> That was the last thing I wanted to ask
14:38 <popey> Ok.
14:38 <popey> I'm happy with the look of the text view as it is
14:38 <popey> line numbers might be nice to switch off.
14:38 <sverzegnassi> they are!
14:38 <sverzegnassi> there's a settings page
14:39 <popey> of course :)
14:40 <popey> (I should have known this)
14:41 <popey> Ok, shall we wrap up then? :)
14:41 <sverzegnassi> yes! :)
14:41 <sverzegnassi> Thank you all!
14:41 <popey> thanks sverzegnassi !
14:42 <popey> good luck with the exam
14:42 <sverzegnassi> thank you!
14:42 <popey> #endmeeting