+
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 Sep 6, 2024

🚀 Description

Adds a filterable attribute to Dropdown. Consumers can use it to force Dropdown to be filterable when there are fewer than 11 options. When there are 11 or more options, Dropdown will be filterable regardless of filterable.

📋 Checklist

  • I have read and followed the Contributing Guidelines.
  • I have added tests to cover new or updated functionality.
  • I have created or updated stories in Storybook to document the new functionality.
  • I have included a changeset with this Pull Request if it adds/updates/removes functionality for consumers.
  • I have scheduled a Design Review for these changes, if one is required.
  • I have followed the ARIA Authoring Practices Guide and/or met with the Accessibility Team to ensure this functionality is accessible.

🔬 How to Test

  1. Navigate to Dropdown in Storybook.
  2. Flip the filterable control to true.
  3. Verify that Dropdown is filterable.

📸 Images/Videos of Functionality

N/A

Copy link

changeset-bot bot commented Sep 6, 2024

🦋 Changeset detected

Latest commit: dabd7fa

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 make-dropdown-optionally-filterable branch from 59505dd to 878622a Compare September 6, 2024 18:22
Copy link
Contributor

github-actions bot commented Sep 6, 2024

@clintcs clintcs force-pushed the make-dropdown-optionally-filterable branch from 878622a to e1f964e Compare September 6, 2024 18:27
},
};

export const Filterable: StoryObj = {
Copy link
Collaborator Author

@clintcs clintcs Sep 6, 2024

Choose a reason for hiding this comment

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

No need for this story now Dropdown has a filterable control.

@clintcs clintcs force-pushed the make-dropdown-optionally-filterable branch 2 times, most recently from 5db5087 to 5841720 Compare September 6, 2024 18:37
@clintcs clintcs force-pushed the make-dropdown-optionally-filterable branch from 5841720 to bab7929 Compare September 6, 2024 18:39
@clintcs clintcs marked this pull request as ready for review September 6, 2024 18:42
Co-authored-by: Tony Ward <8069555+ynotdraw@users.noreply.github.com>
@clintcs clintcs merged commit bd2696e into main Sep 10, 2024
7 checks passed
@clintcs clintcs deleted the make-dropdown-optionally-filterable branch September 10, 2024 13:32
@github-actions github-actions bot mentioned this pull request Sep 10, 2024
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.

3 participants

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