Re: [csswg-drafts] [css-sizing] Percentage sizing section is kind of vague

The Working Group just discussed `[css-sizing] Percentage sizing section is kind of vague`, and agreed to the following resolutions:

* `RESOLVED: Accept the edit in https://github.com/w3c/csswg-drafts/issues/1132#issuecomment-363623845 with a clarification and a hyperlinked term.`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: [css-sizing] Percentage sizing section is kind of vague<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/1132#issuecomment-363623845<br>
&lt;dael> fantasai: Sizing issue was that we...we're asking for review of the comment: https://github.com/w3c/csswg-drafts/issues/1132#issuecomment-363623845<br>
&lt;dael> astearns: There is a test.  There are proposed edits in the comment.<br>
&lt;dael> astearns: Basically you either want a resolution to make the edit or reasons why not or things  to improve?<br>
&lt;dael> fantasai: Yeah. Looking for review.<br>
&lt;dael> fantasai: For margins and padding case I think it's asep issue that we need to discuss.<br>
&lt;dael> fantasai: This text is just about sizing properties. margins is not a sizing property.<br>
&lt;dael> fantasai: For margins and padding we could honor whatever is in  the calc that's n ot the % and treat % as 0. I suspect that would not cause a problem and makes a bit of sense to do if we can.<br>
&lt;dael> fantasai: For sizing properties it's more complicated because you want ot be able tomeasure the content. but for margins and padding there isn't a thing to measure. So we could resolve % against 0 to calc the contining block rather  then ignore the margin entirely.<br>
&lt;dael> franremy: This was  brought up 2 weeks ago. me and dbaron reviewed and I think we were both fine with the proposal. dbaron pointed out you want to link to the sizing properties. I think this is fine. We found more htings tow ork on but it's fine to open a new issue.<br>
&lt;TabAtkins> oh god<br>
&lt;TabAtkins> whoa echo<br>
&lt;dael> dbaron: The one sentence...the one comment is I think containing a % could be two different things. It could be syntatic or mathematic.<br>
&lt;dael> franremy: I think there is a test that covered this. We can clarify the text, but there is a test I think.<br>
&lt;dael> fantasai: Yeah, we need clarification.<br>
&lt;dael> dbaron: I'm  fine given the clarification and hyperlink.<br>
&lt;dael> astearns: Other comments?<br>
&lt;TabAtkins> tabatkins: 0% definitely should count as "containing a percentage".<br>
&lt;dael> astearns: Does anyone object to the change with the clarification and hyperlink?<br>
&lt;fantasai> Discussion of margins is kinda mixed in here : https://github.com/w3c/csswg-drafts/issues/2297<br>
&lt;dael> RESOLVED: Accept the edit in https://github.com/w3c/csswg-drafts/issues/1132#issuecomment-363623845 with a clarification and a hyperlinked term.<br>
&lt;dael> fantasai: I think margin stuff is in issue #2297<br>
&lt;dael> astearns: So franremy please look there and see if you can continue in  that or open a sep issue<br>
</details>


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

Received on Thursday, 8 March 2018 00:31:34 UTC