Bug 303898
| Summary: | [WPT] container-relative units for SVG fails two tests. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Karl Dubost <karlcow> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | ahmad.saleem792, sabouhallawa, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar, WPTImpact |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://wpt.live/css/css-conditional/container-queries/container-units-svglength.html | ||
| Bug Depends on: | 253937 | ||
| Bug Blocks: | |||
Karl Dubost
calc() with container-relative units can be resolved FAIL message: assert_equals: expected 35 but got 0
Can modify value with container-relative units FAIL message: assert_equals: expected "80" but got "40cqw"
https://wpt.fyi/results/css/css-conditional/container-queries/container-units-svglength.html
http://wpt.live/css/css-conditional/container-queries/container-units-svglength.html
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
FIXME: https://searchfox.org/wubkat/rev/c887d10d61230f23a647b1d8fed8a7faa1b022d5/Source/WebCore/svg/SVGLengthValue.cpp#90
```
static Variant<CSS::Number<>, CSS::LengthPercentage<>> createVariantForLengthType(float value, SVGLengthType lengthType)
{
if (lengthType == SVGLengthType::Number)
return CSS::Number<>(value);
if (lengthType == SVGLengthType::Unknown) {
// For unknown types (like container units), fall back to Number
// FIXME: Add support for container units
return CSS::Number<>(value);
}
return CSS::LengthPercentage<>(svgLengthTypeToCSSLengthUnit(lengthType), value);
}
```
Radar WebKit Bug Importer
<rdar://problem/166664174>