ud-annotatrix icon indicating copy to clipboard operation
ud-annotatrix copied to clipboard

User action logger

Open gstark0 opened this issue 7 years ago • 15 comments

As in #189 :) screen shot 2018-01-09 at 22 16 25

@ftyers

gstark0 avatar Jan 09 '18 21:01 gstark0

Note that 2,4 should be a dobj relationship, not nsubj.

jonorthwash avatar Jan 09 '18 21:01 jonorthwash

@jonorthwash Yes, i need to learn a bit about annotating this :)

gstark0 avatar Jan 10 '18 09:01 gstark0

<grzegorzs_> spectre: could you take a look at this? https://github.com/jonorthwash/ud-annotatrix/pull/209
<begiak> [ [WIP] User action logger by gstark0 · Pull Request #209 · jonorthwash/ud-annotatrix · GitHub ]
<spectre> grzegorzs_, looks good
<spectre> but it should be optional
<spectre> e.g. perhaps a button next to the three buttons at the bottom right
<spectre> and i think (to avoid horribly long stuff
<spectre> you could save just the first annotation time
<spectre> and then deltas
<grzegorzs_> spectre: Okay, I will add this, also, id like to improve this, what else would you like it to log?
<spectre> so if the first time is 1045
<spectre> and then the next time is 1048
<spectre> and then the next time is 1100
<spectre> then it would be:
<spectre> AnnoTime=1045,3,52

And then the time at each point would be the sum of the times before.

ftyers avatar Jan 10 '18 09:01 ftyers

@ftyers How about now? Deltas added, button to enable/disable logger also added (toolbar at the top, the icon changes if its enabled or disabled). @jonorthwash There is no dobj on the list :( screen shot 2018-01-10 at 14 05 33

gstark0 avatar Jan 10 '18 13:01 gstark0

I moved the button to the bottom. Icon will be probably changed.

screen shot 2018-01-10 at 17 16 03

gstark0 avatar Jan 10 '18 16:01 gstark0

Icon changed :)

Disabled: screen shot 2018-01-10 at 17 42 09 Enabled: screen shot 2018-01-10 at 17 42 14

gstark0 avatar Jan 10 '18 16:01 gstark0

@gstark0 can you make it so that the enabled icon is the disabled icon but highlighted ? e.g. disabled = blue camera on white background, enabled = white camera on blue background.

ftyers avatar Jan 12 '18 14:01 ftyers

@ftyers Fixed.

gstark0 avatar Jan 12 '18 14:01 gstark0

@gstark0 cool, leave a screenshot and I'll check it out.

ftyers avatar Jan 12 '18 15:01 ftyers

@ftyers screen shot 2018-01-12 at 17 10 47 screen shot 2018-01-12 at 17 10 43

gstark0 avatar Jan 12 '18 16:01 gstark0

Cool, the only thing I wonder now is if we could instead of starting the unixtime from 0 for each token have a comment in the comment block at the beginning with the first time that the sentence was editted and then do offsets from there. But I'm not sure of the pros/cons of that.

ftyers avatar Jan 13 '18 10:01 ftyers

Is there a tooltip for the icon explaining what it does?

jonorthwash avatar Jan 14 '18 23:01 jonorthwash

@jonorthwash sure. But dont Merge it yet, I want to add the features ftyres mentioned.

gstark0 avatar Jan 15 '18 06:01 gstark0

@ftyers Updated. screen shot 2018-01-18 at 19 35 06 screen shot 2018-01-18 at 19 35 24 screen shot 2018-01-18 at 19 35 37

gstark0 avatar Jan 18 '18 18:01 gstark0

@ftyers Here is how it looks like when misc column is not empty: screen shot 2018-01-19 at 15 44 45 screen shot 2018-01-19 at 15 44 54 screen shot 2018-01-19 at 15 45 06 screen shot 2018-01-19 at 15 45 24

gstark0 avatar Jan 19 '18 14:01 gstark0