Closed maxholman closed 6 years ago
Merging #50 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #50 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 2 2
Lines 115 115
Branches 32 34 +2
=====================================
Hits 115 115
Impacted Files | Coverage Δ | |
---|---|---|
src/patch.js | 100% <100%> (ø) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update ef045b0...b6984e5. Read the comment docs.
Should this also handle true
?
I think all browsers will accept setAttribute(name, true)
, but technically it's wrong - the type of the second argument is string
, and according to the specs, the correct way to set a value-less attribute is with the empty string ""
.
@mindplay-dk To what line would you be referring to?
Merged! 🎉
Closes #49
Logic is identical to that of
hyperapp
at time of writing