[motion-1] The computed value of offset-path: path(' ')

[BorisChiou]

Based on the definition of SVG1.1 and SVG2:

The EBNF allows the path data string in the d property to be empty. An empty path data string disables rendering of the path. Rendering is also disabled when the d property has the value none.

So an empty path string is valid. And then what is the expected computed value for this case?

offset-path: path(' ');

If someone calls getComputedStyle(div).offsetPath, should we return path('') or none for offset-path? It seems Blinks return none for now. It looks like the spec doesn't mention this, anyway.

[BorisChiou]

cc @ericwilligers

[ewilligers]

Blink treats and and and all as having path none - example.

I don't see value in maintaining different computed style states for these.

The spec could state that path() with an empty path string serializes as none.

[BorisChiou]

Great. Thanks. This is also for clip-path. I'm also updating Gecko to have none value for clip-path: path(' ').

[emilio]

It seems really odd to serialize it as none but parse it as valid. It seems to me that path('') is a fine path, just with no commands...

[ewilligers]

It seems to me that path('') is a fine path, just with no commands..

It is, that is why in the example it sets the specified value, while path('nonsense') does not. This is about normalization: no commands is equivalent to no path.

[emilio]

But they're not the same, right? offset-path: path('') should establish a containing block and so on. It's just not equivalent to none the same way as transform: translate(0) is not equivalent to transform: none.

[css-meeting-bot]

The CSS Working Group just discussed [motion-1] The computed value of offset-path: path(' '), and agreed to the following:

• RESOLVED: Invalid path strings make the syntax invalid at parse time
• RESOLVED: Empty path is considered invalid

The full IRC log of that discussion

Topic: [motion-1] The computed value of offset-path: path(' ')
github: https://github.com//issues/392
fantasai: I don't know much on this, but it seemed we should resolve and fix.
heycam: The ultimate issue is we have path function which takes string value. string is svg data string.
heycam: Some strings are invalid. What does that mean for validity of path function. If you use invalid path string inside is it all parse as invalid and dropped or is string preserved and invalidity handled later
TabAtkins: If it's valid how to serialize and if not valid clarify that
florian: Precedint in other properties
fantasai: IN grid we throw out what's invalid after parsing
s/Precedint/Precedent/
florian: Same here?
heycam: NO reason not to
florian: No architectural complexity so seems like right thing to do
TabAtkins: Larger question then what's asked in issue. If you have invalid pass it's a separate question. The empty string specifically is a valid path per svg grammer but it disabled rendering of paths. Question is how is it cannonicalized and serialized. In chomre it's serialized as none. All other paths stay as paths but none doesn't so doesn't create containing block
heycam: Surprises me. I think it then does not make sense to serialize as none
TabAtkins: If we decide to treat as invalid that's okay with me and this solves iteself. If we want that we should record it. It's about if empty path is valid or not.
florian: Is it useful?
TabAtkins: No
florian: THen make it invalid ad see if world breaks
TabAtkins: Invalid is probably more consistent with chrome behavior
fantasai: 2 resolutions?
heycam: IN issues it's also mentioned strings with only whitespace. Same?
florian: Suppose so
TabAtkins: Yeah. Regardless of specifics of svg grammar shouldn't treat different
Rossen_: can someone summerize?
TabAtkins: 1) Invalid path strings make the syntax invalid at parse time
Rossen_: Objections?
RESOLVED: Invalid path strings make the syntax invalid at parse time
TabAtkins: 2) The empty string is an invalid path string
florian: With whoever does the edits being careful for all empty types
fantasai: Basically 0 length path
fantasai: 0 length path description
TabAtkins: If we say it's empty we can deal with specifics lather
florian: Empty path not empty strong
s/strong/string
Rossen_: Empty path is considered invalid
Rossen_: Objections?
"a string with no path commands in it", maybe
RESOLVED: Empty path is considered invalid
s/lather/later/
Rossen_: Anything else?