1
78%
master: 78%

Ran 02 Dec 2014 05:36PM UTC

Files 60

Run time 29s

Badge

Embed ▾

Committed 02 Dec 2014 05:16PM UTC coverage: 73.278% (+0.2%) from 73.032%

Job # 1608.1

Build Type

push

travis-ci

Committed by

c0bra

Commit Message

fix(cellNav): Don't setup when directive not there

The cellNav feature was attaching logic to render containers and grid
cells despite the uiGridCellnav directive not being on the parent grid
component. This was causing exception to be thrown during scroll events.

This change looks for the cellNav controller (which is simply
empty) and bails before attaching logic if the controller is not present.

Also added tests to cover the exceptions that were occuring.

Fixes #2128

Coverage Stats

3521 of 4805 relevant lines covered (73.28%)

156.91 hits per line

angular-ui / ng-grid / 1608 / 1
78%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1608.1

angular-ui / ng-grid / 1608 / 1 78% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1608.1

angular-ui / ng-grid / 1608 / 1
78%
master: 78%

README BADGES
x