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

Open addon panel when notification is clicked, and pass new duration option #281

Conversation

ghengeveld
Copy link
Member

@ghengeveld ghengeveld commented Apr 3, 2024

Fixes #264

This uses the new duration and onClick options for addNotification.

Most notifications will auto-close after 8 seconds. The original timeout mechanism is kept in place for backwards compatibility.

When a notification is clicked, the notification is dismissed and the VTA panel is opened/made visible.

To QA, run npx storybook@0.0.0-pr-26696-sha-63254fdd upgrade to get access to the updated notification API.

📦 Published PR as canary version: 1.3.0--canary.281.6e67049.0

✨ Test out this PR locally via:

npm install @chromatic-com/storybook@1.3.0--canary.281.6e67049.0
# or 
yarn add @chromatic-com/storybook@1.3.0--canary.281.6e67049.0

@ghengeveld ghengeveld added the minor Auto: Increment the minor version when merged label Apr 4, 2024
@ghengeveld ghengeveld added the skip-release Auto: Preserve the current version when merged label Apr 4, 2024
@ghengeveld ghengeveld merged commit 1f26873 into main Apr 4, 2024
8 of 9 checks passed
@ghengeveld ghengeveld deleted the ghengeveld/264-make-notification-interactive-so-that-it-can-sho branch April 4, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Auto: Increment the minor version when merged skip-release Auto: Preserve the current version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make notification interactive so that it can show you the changes when you click on it
2 participants