Re: [fxtf-drafts] [motion-1] The computed value of offset-path: path(' ') (#392)

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`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: [motion-1] The computed value of offset-path: path(' ')<br>
&lt;dael> github:  https://github.com/w3c/fxtf-drafts/issues/392<br>
&lt;dael> fantasai: I don't know much on this, but it seemed we should resolve and fix.<br>
&lt;dael> heycam: The ultimate issue is we have path function which takes string value. string is svg data string.<br>
&lt;dael> 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<br>
&lt;dael> TabAtkins: If it's valid how to serialize and if not valid clarify that<br>
&lt;dael> florian: Precedint in other properties<br>
&lt;dael> fantasai: IN grid we throw out what's invalid after parsing<br>
&lt;fantasai> s/Precedint/Precedent/<br>
&lt;dael> florian: Same here?<br>
&lt;dael> heycam: NO reason not to<br>
&lt;dael> florian: No architectural complexity so seems like right thing to do<br>
&lt;dael> 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<br>
&lt;dael> heycam: Surprises me. I think it then does not make sense to serialize as none<br>
&lt;dael> 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.<br>
&lt;dael> florian: Is it useful?<br>
&lt;dael> TabAtkins: No<br>
&lt;dael> florian: THen make it invalid ad see if world breaks<br>
&lt;dael> TabAtkins: Invalid is probably more consistent with chrome behavior<br>
&lt;dael> fantasai: 2 resolutions?<br>
&lt;dael> heycam: IN issues it's also mentioned strings with only whitespace. Same?<br>
&lt;dael> florian: Suppose so<br>
&lt;dael> TabAtkins: Yeah. Regardless of specifics of svg grammar shouldn't treat different<br>
&lt;dael> Rossen_: can someone summerize?<br>
&lt;dael> TabAtkins: 1) Invalid path strings make the syntax invalid at parse time<br>
&lt;dael> Rossen_: Objections?<br>
&lt;dael> RESOLVED: Invalid path strings make the syntax invalid at parse time<br>
&lt;dael> TabAtkins: 2) The empty string is an invalid path string<br>
&lt;dael> florian: With whoever does the edits being careful for all empty types<br>
&lt;dael> fantasai: Basically 0 length path<br>
&lt;dael> fantasai: 0 length path description<br>
&lt;dael> TabAtkins: If we say it's empty we can deal with specifics lather<br>
&lt;dael> florian: Empty path not empty strong<br>
&lt;fantasai> s/strong/string<br>
&lt;dael> Rossen_: Empty path is considered invalid<br>
&lt;dael> Rossen_: Objections?<br>
&lt;heycam> "a string with no path commands in it", maybe<br>
&lt;dael> RESOLVED: Empty path is considered invalid<br>
&lt;astearns> s/lather/later/<br>
&lt;dael> Rossen_: Anything else?<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/fxtf-drafts/issues/392#issuecomment-737579066 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 3 December 2020 00:28:34 UTC