-
Notifications
You must be signed in to change notification settings - Fork 645
SelectPanel: Implement empty state #5142
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
Conversation
…to select-panel-empty-state
🦋 Changeset detectedLatest commit: d3d6875 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks! |
size-limit report 📦
|
| textInputProps={extendedTextInputProps} | ||
| loading={loading || isLoading} | ||
| loadingType={loadingType()} | ||
| // hack because the deprecated ActionList does not support this prop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohhhh nice way to bridge this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jijiji
|
👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks! |
|
👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks! |
…to select-panel-empty-state
|
👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks! |
|
👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks! |
This reverts commit 769a048.
This reverts commit 769a048.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test
Closes https://github.com/github/primer/issues/3922, https://github.com/github/primer/issues/4104
Implementing empty state for SelectPanel. Default message and the ability to customise them when there is no item to show either in the first load or when filtering (Under
primer_react_select_panel_with_modern_action_listFF).Changelog
New
No initial Item
No filtered Item demo
Video description:
Rewatch.Screen.Recording.-.2024-10-24.at.1.55.52.PM.mp4
Changed
Removed
Rollout strategy
Testing & Reviewing
primer_react_select_panel_with_modern_action_listFF must be onMerge checklist