+
Skip to content
This repository was archived by the owner on Oct 14, 2025. It is now read-only.

Conversation

clintcs
Copy link
Collaborator

@clintcs clintcs commented Aug 18, 2025

🚀 Description

Patch

  • Option now dispatches "enabled" and "disabled" events when programmatically enabled or disabled.
  • Option now dispatches "selected" and "deselected" events when programmatically selected or deselected.
  • Options now dispatches a "slotchange" event when the contents of its default slot have changed.

People probably won't need these events because the hope, at least, is that either Menu or Select will work for everyone. But anyone, not just us, should be able to build something like Select on top of Menu. And to do that you need to listen for some or all of these events.

📋 Checklist

  • I have followed the Contributing Guidelines.
  • I have added tests to cover new or updated functionality.
  • I have added or updated Storybook stories.
  • I have localized new strings.
  • I have followed the ARIA Authoring Practices Guide or met with the Accessibility Team.
  • I have included a changeset.
  • I have scheduled a design review.
  • I have reviewed the Storybook and Visual Test Report links below.

🔬 Manual Testing

The tests I added should have us covered.

Copy link

changeset-bot bot commented Aug 18, 2025

🦋 Changeset detected

Latest commit: 3a10a89

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@crowdstrike/glide-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@clintcs clintcs force-pushed the expose-option-and-options-private-events branch 4 times, most recently from 5c66818 to b9efb1e Compare August 18, 2025 17:32
@clintcs clintcs force-pushed the expose-option-and-options-private-events branch from b9efb1e to 3a10a89 Compare August 18, 2025 17:34
summary: 'method',
detail: `
// "slotchange" is useful when you are abstracting over Menu to add functionality to it. See Select's
// use of this event for an example.
Copy link
Collaborator Author

@clintcs clintcs Aug 18, 2025

Choose a reason for hiding this comment

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

Select doesn't exist yet. But it will shortly!

@clintcs clintcs marked this pull request as ready for review August 18, 2025 17:41
@clintcs clintcs added this pull request to the merge queue Aug 18, 2025
Merged via the queue into main with commit 5589e6b Aug 18, 2025
32 checks passed
@clintcs clintcs deleted the expose-option-and-options-private-events branch August 18, 2025 18:44
@github-actions github-actions bot mentioned this pull request Aug 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载