421
85%

Ran 25 Jun 2017 04:21PM UTC

Jobs 2

Files 8

Run time 41s

Badge

Embed ▾

pending completion

Build # 421

Build Type

Pull #441

travis-ci

Committed by

web-flow

Commit Message

[fixed] 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: [fixed] improvements on setAppElement...

Run Details

168 of 283 branches covered (59.36%)

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

301 of 352 relevant lines covered (85.51%)

39.31 hits per line

New Missed Lines in Diff

Lines	Coverage	∆	File
1	93.02	3.37%	src/helpers/ariaAppHider.js

Uncovered Existing Lines

Lines	Coverage	∆	File
1	93.02	3.37%	src/helpers/ariaAppHider.js
7	92.31	-0.08%	src/components/Modal.js

Jobs

ID	Job ID	Ran	Files	Coverage
1	421.1	25 Jun 2017 04:21PM UTC	8	85.51	Travis Job 421.1
2	421.2	25 Jun 2017 04:21PM UTC	8	85.51	Travis Job 421.2

Source Files on build 421

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

reactjs / react-modal / 421
85%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Uncovered Existing Lines

Jobs

Source Files on build 421

reactjs / react-modal / 421 85%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Uncovered Existing Lines

Jobs

Source Files on build 421

reactjs / react-modal / 421
85%

README BADGES
x