5160
54%
master: 97%

Ran 13 Mar 2015 11:22AM UTC

Jobs 1

Files 51

Run time 37s

Badge

Embed ▾

pending completion

Build # 5160

Build Type

push

travis-ci

Committed by

csillag

Commit Message

Fix sidebar's focusing on annotation cards

When the user moves the mouse over a highlight in the document,
the corresponding annotation card (in the sidebar) is supposed
to be highlighted, to indicate focus.

Our mechanism for this was broken in 3 different ways:

1. AnnotationUI service's focusAnnotations() method was working
   with annotation IDs. However, annotations being created
   right now don't yet have ID, only tags. So we should be working
   with tags instead. (Like we are, in many other cases
   when cross-frame communication communication is done
   around annotations.)

   I fixed that, and also updated the corresponding tests.

2. In controllers.coffee, when we were trying to check if a given
   annotation has focus, we were using the tag value, while on
   the other hand, it was not the tag, but the id that was put
   into the map. Of course this didn't work.

   Since now we are consistently using the tag, no change
   was needed here.

3. We are putting $$tag:true pairs into the map, but when
   checking for membership, the `in` operator was used.
   That works for lists, but not for maps. For maps, we simply
   have to read out the value belonging to the given key,
   and see if it's true.

After fixing these problems, the feature works again.

Run Details

1231 of 2265 relevant lines covered (54.35%)

0.54 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	5160.1 (BROWSERIFY_PIPE_BIN=bin/browserify-pipe)	13 Mar 2015 11:22AM UTC	0	54.35	Travis Job 5160.1

Source Files on build 5160

Detailed source file information is not available for this build.

hypothesis / h / 5160
54%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5160

hypothesis / h / 5160 54% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5160

hypothesis / h / 5160
54%
master: 97%

README BADGES
x