Re: [csswg-drafts] [css-ui] DOM/Box structure for appearance:base-select (#10380)

The CSS Working Group just discussed `[css-ui] DOM/Box structure for appearance:base-select`, and agreed to the following:

* `RESOLVED: <select> internals can be represented with shadow DOM (but shadow DOM support is not required as long as the behavior is the same)`
* `RESOLVED: Accept shadow dom structure as proposed initial comment in the issue, except for the svg part and naming of pseudo elements. issues to be opened for those`

<details><summary>The full IRC log of that discussion</summary>
&lt;emilio> scribenick: emilio<br>
&lt;emilio> chrishtr: we can mark as needs edit and once that's done that'd get review and get closed<br>
&lt;emilio> annevk: sounds good<br>
&lt;dbaron> (above was about the previous item on the agenda)<br>
&lt;emilio> jarhar: That's a proposal for base-select behavior and DOM structure<br>
&lt;emilio> ... higher level questions would be whether it'd be ok to use shadow dom for this<br>
&lt;emilio> ... the other thing would be that in chromium elements inside the shadow root for the base and auto appearance<br>
&lt;gregwhitworth> q?<br>
&lt;emilio> ... and switch based on the computed value<br>
&lt;emilio> ... wonder if that's acceptable<br>
&lt;emilio> annevk: I think shadow roots are fine for this, we have the precedent of &lt;details><br>
&lt;fantasai> +1 to avoiding a true dependency on the shadow DOM<br>
&lt;emilio> ... in theory an implementation could not use shadow root<br>
&lt;emilio> q+<br>
&lt;emilio> jarhar: that sounds promising<br>
&lt;emilio> ... it sounds like we can define this as a shadow root<br>
&lt;emilio> masonf: are we doing resolutions?<br>
&lt;fantasai> PROPOSED: &lt;select> internals can be represented with shadow DOM<br>
&lt;masonf> +1<br>
&lt;dandclark> +1<br>
&lt;gregwhitworth> emilio: I wanted to clarify fantasai point, I think she's fine defining in terms of shadow DOM but in theory you can implement it with something else<br>
&lt;gregwhitworth> fantasai: yeah, that's exactly right we should define it using shadow but it's ok if the implementation does not use shadow DOM<br>
&lt;emilio> emilio: sounds good<br>
&lt;gregwhitworth> chrishtr: they need to end up with the same interoperable result<br>
&lt;emilio> chrishtr: it's not observable that there's a shadow root right?<br>
&lt;masonf> shady-dom<br>
&lt;emilio> jarhar: I think that's correct<br>
&lt;emilio> chrishtr: we can put a non-normative note in the spec about it being possibly implemented with other technology<br>
&lt;fantasai> PROPOSED: &lt;select> internals can be represented with shadow DOM (but shadow DOM support is not required as long as the behavior is the same)<br>
&lt;emilio> gregwhitworth: so fine to go with Elika's resolution<br>
&lt;emilio> *?<br>
&lt;masonf> +1<br>
&lt;keithamus> +1<br>
&lt;emilio> RESOLVED: &lt;select> internals can be represented with shadow DOM (but shadow DOM support is not required as long as the behavior is the same)<br>
&lt;emilio> jarhar: I want to also go through the slots and so on<br>
&lt;emilio> gregwhitworth: Should we resolve that in this issue or move that to a different one?<br>
&lt;emilio> annevk: the one concern from our side is about the `multiple` attribute<br>
&lt;emilio> ... without that we can agree to this<br>
&lt;emilio> ... but with `multiple` one of the slots ends up not being there or rendered<br>
&lt;emilio> jarhar: we can change the shadow DOM for the `multiple` attribute<br>
&lt;emilio> ack emilio<br>
&lt;gregwhitworth> ack fantasai<br>
&lt;emilio> jarhar: so that there's a shadow DOM but can be completely different<br>
&lt;chrishtr> q+<br>
&lt;emilio> fantasai: currently in html we combine two things, if you can select one it's a popup and otherwise it's an embedded control<br>
&lt;gregwhitworth> q+<br>
&lt;masonf> q+<br>
&lt;emilio> ... I think we can split that into multiple axes<br>
&lt;masonf> that's the multiple and size attributes, respectively.<br>
&lt;annevk> (modulo iOS where it's a little different from that even)<br>
&lt;gregwhitworth> ack chrishtr<br>
&lt;jarhar> q?<br>
&lt;lwarlow> (and android)<br>
&lt;emilio> ... so that we can make a popup where you can select multiple attributes, and you can have an embedded thing where you can select just one<br>
&lt;emilio> chrishtr: all those things can be figured out but can we leave those not defined yet?<br>
&lt;emilio> ... and define just the non-multiple case for now?<br>
&lt;emilio> fantasai: I think we might need to come back and change things once we tackle the others<br>
&lt;gregwhitworth> ack gregwhitworth<br>
&lt;gregwhitworth> ack masonf<br>
&lt;emilio> chrishtr: we can take an action item to dig into that to make sure that changes might not be needed<br>
&lt;emilio> masonf: there's `multiple` and `size` which theoretically create those two axes<br>
&lt;emilio> ... behavior is a bit different in mobile vs. not, I think there are a lot of issues with `multiple` without that<br>
&lt;gregwhitworth> q+<br>
&lt;emilio> ... so I think we need feature detection but I'd like not to block on that<br>
&lt;lwarlow> q+<br>
&lt;emilio> fantasai: I think that'd be surprising, I'd like multiple to be defined for styleable select<br>
&lt;emilio> ... I think you can't just ignore the semantics of `multiple`<br>
&lt;gregwhitworth> ack gregwhitworth<br>
&lt;emilio> masonf: I was thinking of just forcing `auto` rather than ignoring `multiple`<br>
&lt;lwarlow> q-<br>
&lt;emilio> fantasai: I think I worry about site issues once we start making changes there, I think it's solvable and we should fix it<br>
&lt;emilio> chrishtr: agree on solvable, I don't think it blocks resolving on jarhar's proposal for the single case<br>
&lt;emilio> fantasai: I think the structure makes sense except the svg<br>
&lt;emilio> ... because it doesn't respect writing-mode<br>
&lt;emilio> ... in respect on the names of pseudos we want to be consistent with other stuff we expect to do elsewhere<br>
&lt;jarhar> q?<br>
&lt;chrishtr> proposed: "accept shadow dom structure as proposed initial comment in the issue, except for svg part"<br>
&lt;emilio> ... but over-all structure makes sense otherwise<br>
&lt;fantasai> s/to do elsewhere, so probably should discuss separately/<br>
&lt;emilio> jarhar: for pseudo names I agree they're not great<br>
&lt;chrishtr> "pseudo element names to be decided later"<br>
&lt;emilio> ... there's another issue for that<br>
&lt;emilio> ... as for the SVG I agree, there's a similar discussion for checkmarks<br>
&lt;emilio> ... we moved to a unicode character<br>
&lt;emilio> ... there's a down arrow thing<br>
&lt;chrishtr> q+<br>
&lt;emilio> ... do you think using it would be reasonable<br>
&lt;emilio> fantasai: if you use disclosure-open then it takes care of those issues<br>
&lt;gregwhitworth> ack chrishtr<br>
&lt;dbaron> s/disclosure-open/the disclosure-open counter style/<br>
&lt;emilio> chrishtr: suggestion is to make some forward progress on this. We can discuss checkmarks / svg / names later<br>
&lt;emilio> gregwhitworth: I agree, follow-up issues for glyph and pseudo-elements makes sense<br>
&lt;chrishtr> "accept shadow dom structure as proposed initial comment in the issue, except for svg part and naming of pseudo elements. issues to be opened for those"<br>
&lt;emilio> fantasai: should probably be one issue for the pseudos and such, we should decide them together<br>
&lt;jarhar> i already created an issue for pseudo elements here: https://github.com/w3c/csswg-drafts/issues/10462<br>
&lt;masonf> +1<br>
&lt;gregwhitworth> q?<br>
&lt;chrishtr> "accept shadow dom structure as proposed in the initial comment in the issue, except for svg part and naming of pseudo elements. issues to be opened for those"<br>
&lt;emilio> fantasai: inside the button one of the elements is a `&lt;span>` and another a `&lt;div>`, why?<br>
&lt;gregwhitworth> RESOLVED: Accept shadow dom structure as proposed initial comment in the issue, except for the svg part and naming of pseudo elements. issues to be opened for those<br>
&lt;emilio> jarhar: good question, maybe I was relying on block vs inline layout, but I might be overriding display anyways<br>
</details>


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


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

Received on Thursday, 27 June 2024 15:25:53 UTC