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

Translating storybook stories to new API #50127

Merged
merged 4 commits into from
Feb 9, 2023

Conversation

hannahbergam
Copy link
Contributor

@hannahbergam hannahbergam commented Feb 7, 2023

This PR translates three storybook files of shared components to the new API, including Dialog and BaseDialog!

QuickActionsCell - I added a div with a styling so that you can actually see the popup when you click to view the options. Example here:
Screen Shot 2023-02-07 at 6 26 48 PM

BaseDialog has several different potential views. This makes for a great storybook! Here is one example (after clicking to open):
Screen Shot 2023-02-07 at 6 40 55 PM

And finally, the Dialog component. This one was fun- there are so many great examples are they're all quite different! Here is the full width one:
Screen Shot 2023-02-07 at 6 50 36 PM

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

component: Dialog
};

/*
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried to preserve the instructions in a comment, so that a developer would still be able to read them!

/>
);

export const NoFooter = () => (
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Though I didn't add the descriptions when all the messaging comes through with a descriptive name:)

@hannahbergam hannahbergam requested a review from a team February 8, 2023 02:57
Copy link
Contributor

@bethanyaconnor bethanyaconnor left a comment

Choose a reason for hiding this comment

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

🎉

@hannahbergam hannahbergam merged commit d02d85f into staging Feb 9, 2023
@hannahbergam hannahbergam deleted the hbergam/quick-actions-story branch February 9, 2023 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants