Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add <selectlist> draft feature #581

Merged
merged 2 commits into from
May 6, 2024
Merged

Add <selectlist> draft feature #581

merged 2 commits into from
May 6, 2024

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented Feb 3, 2024

No description provided.

@foolip
Copy link
Collaborator Author

foolip commented Feb 3, 2024

@ddbeck this is an interesting case. https://caniuse.com/selectlist already exists and there's excitement around this feature, but it doesn't have a spec yet, and I don't think we can be 100% sure the eventually shipping feature will even be a <selectlist> element. This is more bleeding edge than we've dealt with in web-features before. What to do?

@captainbrosset
Copy link
Contributor

I don't think we can be 100% sure the eventually shipping feature will even be a <selectlist> element

Correct, there's been discussions about changing this to extend the <select> element instead. This is still very much in flux. If anything, this should probably be named something like "Stylable select".

@@ -0,0 +1,3 @@
name: "<selectlist>"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The explainer has now been changed to <select> instead of <selectlist>: https://open-ui.org/components/selectlist/

@ddbeck
Copy link
Collaborator

ddbeck commented Feb 5, 2024

I can see that we might want to include not-yet implemented features, but I worry about maintenance and data quality issues.

BCD's policy is to not include features without implementations. One benefit of that approach is that proposals that go nowhere (or implementations that are removed) don't live on as zombie features. Instead, lack of compat data is a clear signal to either add it (in the case of real features) or dispense with content (in the case of unimplemented features).

If we're going to do something different, then I'd like to see an obvious mechanism to dispense with unmaintained features (e.g., an editorial field with an expiration date could allow us to automatically generate removal PRs, if there's no change in the support status).

I'd also like to ensure that we have some kind of there there. I'd prefer specs, but we might have an allowlist for issue trackers to qualify such primordial features (e.g., WHATWG, tc39/proposals, etc.)

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label May 2, 2024
@foolip foolip changed the title Add <selectlist> feature Add <selectlist> draft feature May 3, 2024
@foolip foolip marked this pull request as ready for review May 3, 2024 12:31
@foolip
Copy link
Collaborator Author

foolip commented May 3, 2024

I have turned this into a draft feature (and no longer a draft PR, yeah, that might get confusing).

@foolip foolip requested a review from captainbrosset May 3, 2024 12:32
@foolip foolip enabled auto-merge (squash) May 6, 2024 11:30
@foolip foolip merged commit cc6bd8c into main May 6, 2024
3 checks passed
@foolip foolip deleted the selectlist branch May 20, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature definition Creating or defining new features or groups of features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants