413
85%

Ran 25 Jun 2017 04:54AM UTC

Jobs 2

Files 8

Run time 30s

Badge

Embed ▾

pending completion

Build # 413

Build Type

Pull #441

travis-ci

Committed by

web-flow

Commit Message

improvements on setAppElement...

There was some pitfalls on how `setAppElement` work.

If your <script /> was in <head />, there was a change that it tries
to use `document.body` that is not yet ready.

Another one was using an selector string that does not find any
elements, causing it to try to perform all call on `null`.

This patch can also help if you want to do server-side rendering,
but this was not tested and, perhaps, it's better to use this function
correctly.

Pull Request Pull Request #441: improvements on setAppElement...

Run Details

170 of 286 branches covered (59.44%)

16 of 16 new or added lines in 2 files covered. (100.0%)

297 of 347 relevant lines covered (85.59%)

39.85 hits per line

New Missed Lines in Diff

Lines	Coverage	∆	File
1	94.59	4.94%	src/helpers/ariaAppHider.js

Uncovered Existing Lines

Lines	Coverage	∆	File
4	92.39	0.0%	src/components/Modal.js

Jobs

ID	Job ID	Ran	Files	Coverage
1	413.1	25 Jun 2017 04:54AM UTC	8	85.59	Travis Job 413.1
2	413.2	25 Jun 2017 04:54AM UTC	8	85.59	Travis Job 413.2

Source Files on build 413

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line	Branch Hits	Branch Misses

reactjs / react-modal / 413
85%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Uncovered Existing Lines

Jobs

Source Files on build 413

reactjs / react-modal / 413 85%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Uncovered Existing Lines

Jobs

Source Files on build 413

reactjs / react-modal / 413
85%

README BADGES
x