Re: [csswg-drafts] [css-values] Should viewport units still depend on scrollbar width for overflow:scroll?

The Working Group just discussed `Should viewport units still depend on scrollbar width for overflow:scroll?`.

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: Should viewport units still depend on scrollbar width for overflow:scroll?<br>
&lt;tantek> because:  &lt;dael> jensimmons: I'm fine punting to next week.<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/1766<br>
&lt;dael> astearns: We're waiting on impl feedback?<br>
&lt;dael> TabAtkins: Yes, I'm getting ?? to comment.<br>
&lt;zcorpan_> (Could css-meeting-bot remove the label when it's commenting?)<br>
&lt;astearns> it does when we resolve something<br>
&lt;dael> dbaron: I reised this bc gecko was the only engine that impl when we decided this years ago. It's apperently harder to do on silo. I think it's reasonable on their part.<br>
&lt;dael> myles: If all browers except Moz ones don't support and MOz wants to remove it seems clear<br>
&lt;dael> TabAtkins: Without this feature there's alot of cases where viewport is broken. THere are plent that do, but I suspect people are avoiding these thigns. You cannot get 100vw to be the size of the screen. THere's usually a scrollbar and 100vw is too big.<br>
&lt;dael> myles: The desc says it's only on sroll not auto<br>
&lt;dbaron> s/silo/stylo/<br>
&lt;dael> TabAtkins: We decided viewport should be resolved at computed. For overflow auto we decided to ignore scrollbar, but w wanted to pay attention when it's there. overflow:scroll causes a scrollbar so we set it to take scrollbar into account in that case.<br>
&lt;dael> myles: Overflow:Scroll on root is not common case<br>
&lt;Florian> q+<br>
&lt;dael> TabAtkins: I agree. But when your scrollbar appears your items to 100vw will overflow horz and cause a scrollbar. The way we decided to let tha tbe solve dyou can set overflow scroll on it explicitly so it fits. Without this viewport units are just broken. The only time they add to the viewport is when you have no scrollbar which is uncommon<br>
&lt;dael> fremy: You still need to know size of scrollbar. It's not a static number.<br>
&lt;dael> Rossen: We're talking about uint value. I'm with TabAtkins. If in the case of overflow: scroll when scrollbar takes spec away from viewable viewport current spec mandates the correct behavior from user PoV. viewport should resolve to are-scrollbars.<br>
&lt;dael> Rossen: Changing spec for interop and going aaginst what makes sense isn't best.<br>
&lt;astearns> ack Florian<br>
&lt;dael> Florian: If we don't drop the thing should we extend it?  Even if it's auto?<br>
&lt;dael> TabAtkins: Prob should. Assuing whatever element applies to root scroller. If that works and we define how that would help.<br>
&lt;dael> Rossen: I would argue against that. THe elemnt causing the scrollbar could be defined in vh unit which causes scrollbar and introduces a dependency. IN the case of overflow scroll on the viewport we only need to add computed value and resolve overflow for the root. From there on decide what he value will be.<br>
&lt;dael> fantasai: No one is suggesting auto for scrollbar. There's a scrollbar gutter property that adds extra space.<br>
&lt;dael> Rossen: I was reading the minutes from Florian about perhaps extending to auto.<br>
&lt;Florian> https://drafts.csswg.org/css-overflow-4/#scollbar-gutter-property<br>
&lt;fantasai> Florian was asking about scrollbar-gutte rproperty<br>
&lt;dael> TabAtkins: It's only when you ahve a predicatble value to auto<br>
&lt;fantasai> s/should we extend it/should we extend it to handle the scrollbar-gutter property/<br>
&lt;dael> Florian: Even though you get that you get a predictable size with space reserved but not scrollbar so maybe that shouldn't count. I don't know.<br>
&lt;astearns> q+<br>
&lt;dael> Rossen: Let's assume simple l to r. Whatever an element with abspos top 0 right 0 wherever that positions to shoudl be extent of vh<br>
&lt;dael> TabAtkins: No because then by default auto changes value of vw unit and that defets the computed value time purpose<br>
&lt;dael> Rossen: No in the cae of the gutter we're talking about the prop reserving space for gutter<br>
&lt;dael> TabAtkins: If gutter is predictable yet<br>
&lt;dael> Rossen: That's what I'm talking about. So if abspos items are positioned to gutter this should be extent of vh as well?<br>
&lt;dael> Florian: I don't think we explicitly define it may fall out of the wording, but I don't htink that was concious decision.<br>
&lt;dael> Florian: I don't think we decided on that<br>
&lt;dael> astearns: Is that enough on the gutter sidebar?<br>
&lt;Rossen> q?<br>
&lt;dael> Florian: I think we need to come back, but not for this call.<br>
&lt;dael> astearns: I want to go back to the discussion before gutters where...I agree that vw should take the scrollbar into account where it exists. But the one piece of impl feedback on that is it's really annoying to have to layout arbitrary things when you gain a scrollbar. Would it make sense to define auto same as scroll case for vw? It's always taking into case poss scrollbar?<br>
&lt;dael> TabAtkins: If you have overflow: hidden on root vw doesn't stretch all the way across the screen.<br>
&lt;dael> Florian: Just auto.<br>
&lt;dael> TabAtkins: No, still hidden. Just changing auto doesn't fix dbaron problem. He didn't want size of vw change based on somebody updating [missed]<br>
&lt;dbaron> I think there were actually 2 issues.<br>
&lt;dael> TabAtkins: Change at all is the worry. Main responce is this is no different hen m unit. You change the font and everything below has to change. Only more complicated is that sometimes body can set viewport. So you can effect one element up.<br>
&lt;Rossen> dbaron, can you pls summarize?<br>
&lt;dbaron> One was that dynamic changes to 'overflow' are hard, but that the other is that in Gecko, we need to depend on layout to find out what the scrollbar width is.<br>
&lt;dael> TabAtkins: Aside from that one element jump which is very rare and not important for purpose of tree expence this is identical to setting font size.<br>
&lt;dael> TabAtkins: You dont' need layout this is computed value time<br>
&lt;dael> dbaron: Second thing that made it had is in gecko it's not ready.<br>
&lt;dael> dbaron: [missed]<br>
&lt;dael> dbaron: I'm not even sure it worked rilably. I think it worked most of the time because usually we constructed scrollbars first.<br>
&lt;dael> TabAtkins: Is that b/c scrollbar width is controllable in FF?<br>
&lt;dael> dbaron: It depends on too many OS dependant and that code exists in scrollbar layout.<br>
&lt;dael> TabAtkins: Yeah.<br>
&lt;dael> dbaron: It's possible that the number of things to test is 2 or 3, but that's not the way it was written<br>
&lt;Rossen> wouldn't webkit-scrollbar {width} have the same effect?<br>
&lt;dael> astearns: We're at time. i'm hearing the desire to keep spec behavior as-is, but we'll need other impl to make the changes. WE should continue impl feedback in the issue.  We're out of time today, we'll talk next week.<br>
&lt;TabAtkins> s/Yeah./So maybe just moving the code up and piping the data down into layout instead./<br>
</details>


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

Received on Wednesday, 30 August 2017 17:02:45 UTC