On Fri, Oct 11, 2024 at 5:33 AM Joey Arhar <jar...@chromium.org> wrote:
> Contact emailsjar...@chromium.org > > Explainerhttps://open-ui.org/components/customizableselect > > Specificationhttps://github.com/whatwg/html/pull/10557 > This PR is still getting pretty active review comments. At least some of them seem like they'll impact the behavior. Do you think it would be reasonable to wait for the PR to settle before approving? I don't think we necessarily need to wait for formal approval or merging, but it seems like right now normative steps (like "Reconstruct the active formatting elements, if any") are being added and removed within the last 2 hours. > > > Summary > > This change makes the HTML parser allow additional tags in <select> > besides <option>, <optgroup>, and <hr>. This change is in support of the > customizable <select> feature but is being shipped first because it can be > done separately and has some compat risk which I'd like to get feedback on. > Customizable select explainer: > https://open-ui.org/components/customizableselect/ I did a compat > analysis and determined that the vast majority of sites which would see the > effects of the parser changes would not have their behavior changed. More > details here: https://github.com/whatwg/html/issues/10310 If there are > major issues with this change, I will reassess and make adjustments to the > parser as needed. > > > Blink componentBlink>HTML>Parser > <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EHTML%3EParser> > > TAG reviewNone > > TAG review statusPending > I suspect this doesn't need a separate TAG review from the general customizable <select> one. But could you update ChromeStatus and this thread to link to the customizable select TAG review? > > > Risks > > > Interoperability and Compatibility > > I believe there is low interop risk because other vendors are reviewing > the proposal in WHATWG and are not objected to changing the parser behavior > here. There is a compat risk if websites are relying on the old parser > behavior. As I mentioned in the main description, I have done an analysis > and based on the websites I have investigated I believe there is low risk. > If we encounter issues then I will adjust the parser as needed. > > > *Gecko*: Positive of experimenting: > https://github.com/whatwg/html/issues/10310#issuecomment-2189178702 > https://github.com/mozilla/standards-positions/issues/1086 is probably the better link, which looks like "No signal but leaning positive pending confirmation". > > > *WebKit*: No signal ( > https://github.com/WebKit/standards-positions/issues/414) > > *Web developers*: No signals ( > https://github.com/mozilla/standards-positions/issues/1086) > > *Other signals*: > > WebView application risks > > Does this intent deprecate or change behavior of existing APIs, such that > it has potentially high risk for Android WebView-based applications? > > None > > > Debuggability > > None > > > Will this feature be supported on all six Blink platforms (Windows, Mac, > Linux, ChromeOS, Android, and Android WebView)?Yes > > Is this feature fully tested by web-platform-tests > <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> > ?Yes > The spec PR says "I still need to implement html5lib test changes". Did that get done? > > > Flag name on chrome://flagsSelectParserRelaxation > > Finch feature nameSelectParserRelaxation > > Requires code in //chrome?False > > Tracking bughttps://crbug.com/335456114 > > Estimated milestones > Shipping on desktop 130 > DevTrial on desktop 128 > Shipping on Android 130 > DevTrial on Android 128 > Shipping on WebView 130 > Are these accurate? The branch point for 130 has passed a while ago, and it sounds like the behavior is still changing (per the 2-hours-ago discussions on the spec PR). > > Anticipated spec changes > > Open questions about a feature may be a source of future web compat or > interop issues. Please list open issues (e.g. links to known github issues > in the project for the feature specification) whose resolution may > introduce web compat/interop risk (e.g., changing to naming or structure of > the API in a non-backward-compatible way). > None > > Link to entry on the Chrome Platform Status > https://chromestatus.com/feature/5145948356083712?gate=5114873999261696 > > This intent message was generated by Chrome Platform Status > <https://chromestatus.com/>. > > -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAK6btw%2B2u4-MY3c2x5uci_dOJWw8UdDNGfr_BiSfNESQwmPh5w%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAK6btw%2B2u4-MY3c2x5uci_dOJWw8UdDNGfr_BiSfNESQwmPh5w%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra_vqc6HvYXfPp0B0528Dq40QGBPUqwK%3DttZ_p8fJxG%2BbA%40mail.gmail.com.
