Re: [csswg-drafts] Definiteness of flex items' main size depend on flex-basis's definiteness

The CSS Working Group just discussed `Definiteness of flex items' main size depend on flex-basis's    definiteness`, and agreed to the following resolutions:

* `RESOLVED: Allow % to resolve, re-open issue if dholbert or others want to rediscuss.`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: Definiteness of flex items' main size depend on flex-basis's    definiteness<br>
&lt;dael> github topic: https://github.com/w3c/csswg-drafts/issues/1679<br>
&lt;dael> Bert: I think gregwhitworth wanted to look at code he had to compare.<br>
&lt;dael> gregwhitworth: Is anybody from FF on?<br>
&lt;dael> dbaron: I don't know if I'll be helpful.<br>
&lt;dael> gregwhitworth: I put my information in the notes. I sat with our dev. We could out the same thing as dholbert. We haven't impl this aspect. I pushed him tos ee if this is right approach. Dholbert seemed to say he thought it was the right approach.<br>
&lt;dael> gregwhitworth: WE do because everything is auto we do layout and then a second pass to resolve %. That's what authors also expect. I'm wondering if we should remove the constraing on #2 of 1.8. If you set definte flex basis and fixed height it does the extra work. I see a lot of authors doing this though it's not what they're expecting.<br>
&lt;dael> fantasai: Bigger argument is we have introp on 2 impl and the authors expect that behavior. Spec says something else so we should change spec.<br>
&lt;dael> gregwhitworth: And what dholbert posted to the list, we originally said hey performance. It's odd for authors to know that to set flex-basis to 0 is what you need to do for chorome to look like edge and ff.<br>
&lt;fremy> fwiw I discussed this with gregwhitworth yesterday, and I totally +1 his request to remove the constraints on #2<br>
&lt;dael> gregwhitworth: Let's just removet he constraints of #2 under 9.8<br>
&lt;dael> fantasai: Reasonable to me.<br>
&lt;dael> Bert: Me too, but I'm not expert. Other opinions?<br>
&lt;dael> dbaron: I'm not sure how well performance bugs get tied back to this. I'm not sure if we heard there were performance bugs.<br>
&lt;dbaron> s/heard/would have heard/<br>
&lt;dbaron> s/there/if there/<br>
&lt;dael> gregwhitworth: That's a valid arguments. I feel like we're going to argue a theory. Now that we have grid we'll be able to have less nested flexes is my guess so I'm not as worried about the circular problem.<br>
&lt;dael> fantasai: IN grid % always resolved b/c grid sizing creates sizes for the tracks. Conseptually grid algo resolves the eq. code the way FF and Edge do flex right now.<br>
&lt;dael> Bert: I'm hearing, fantasai , that there are impl that do and some that don't do the layout based sizing?<br>
&lt;dael> gregwhitworth: Chrome and I believe I saw webkit and blink would have to change. As we stated the people on the thread aren't on the call.<br>
&lt;gsnedders> FF and Edge agree, Chrome and Safari agree (but are one impl now again)<br>
&lt;dael> Bert: Are we confident enough to decide? Do we need more input?<br>
&lt;gsnedders> WebKit's implementation of flex was literally copied over from Blink a few months ago<br>
&lt;dael> fantasai: Christian is away for the next couple months. We can ask dholbert that we want this resolution and ask for approval. From dholbert and whomever people want to tag.<br>
&lt;dael> Bert: Sounds like a good way forward. If we document it in the issue is that enough? Do we need to contact personally?<br>
&lt;dael> gregwhitworth: I say we just resolve since we discussed it. We can rediscuss if they want to get on a call<br>
&lt;dael> Bert: Okay.<br>
&lt;dael> Bert: wording for proposed resolution:<br>
&lt;dael> gregwhitworth: It's what I said in #2. Basically re-wirte #2 of section 9.8 to % would resolve.<br>
&lt;dael> fantasai: Allow % to resolve, re-open issue if dholbert or others want to rediscuss.<br>
&lt;dael> Bert: Okay, that's the proposal. Objections?<br>
&lt;gregwhitworth> s/it's what I said in #2/it's what I said in the issue<br>
&lt;dael> RESOLVED: Allow % to resolve, re-open issue if dholbert or others want to rediscuss.<br>
</details>


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

Received on Wednesday, 23 August 2017 16:27:25 UTC