Fixes#7152, see also #7288
--foo:; used to be an invalid CSS custom property value, while -foo: ; was valid. By collapsing the whitespace in these declaration values, we were breaking scenarios where an empty custom property was desired. The spec was updated to trim whitespace and treat these values identically, but Chromium browsers still treat --foo; as invalid. This was recently fixed and will be released in Chrome 99, but this would still be a good fix to maintain backwards compatibility.
* Strip out whitespace inside each when it has an animation
* remove accidentally committed file
* lint
* add test to validate no error
* update changelog
Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Co-authored-by: tanhauhau <lhtan93@gmail.com>
Co-authored-by: Tan Li Hau <tanhauhau@users.noreply.github.com>
- deduplicate type name: Is now "StyleDirective"
- Don't transform value array to template literal in the AST phase but in the compiler phase. This ensures other tools know what the raw output was and that start/end positions are available
* add Style node, interfaces
* style-directives: add parser and runtime test
* style-directives: push styles in to styles array on Element
* style-directives: minimal ssr work
* style-directives: ssr add_styles
* style-directive: tests
* style-directives: work in progress
* obviously incorrect hard-coded color
* tweak
* style directive interface
* style-directives: get text from info in Style node
* style-directives: add_styles func in ElementWrapper
* style-directives: ssr rendering
* handle text-only style directive in tag.ts
* style-directives: handle spread styles in ssr
* style-directives: more parser tests
* style-directives: more inline tests
* style-directives: remove solo tests
* style-directives: cleanup
* style-directives: tweak spread ssr internal
* style-directives: push updater into update chunks; add dynamic test;
* remove .solo
* check for dynamic dependencies before adding update chunk
* add test of multiple styles; remove null styles;
* style-directives: docs; more tests of multiple styles
* style-directives: use camelcase
* style-directives: cleanup
* style-directives: fix mustache template case with template literal
* style-directives: use ternary
* style-directives: linting
* style-directives: remove "text" from interface
* style-directives: actually, remove StyleDirective interface entriely
* add more test, fix test for ssr
* fix lint and tidy up
* add test for css variables
* fix linting errors
Co-authored-by: pmurray73 <pmurray73@bloomberg.net>
Co-authored-by: Tan Li Hau <lhtan93@gmail.com>