-
Notifications
You must be signed in to change notification settings - Fork 889
feat: add a divider after Account menu item #1927
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
feat: add a divider after Account menu item #1927
Conversation
I didn't find any Jest tests that failed, do we have any snapshot tests for our React components, or any other sort of screenshots that need an update? The Storybook for |
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.
Thanks for doing this!!!
That's a good callout. I think adding a story with the UserDropdown open would be helpful. A quick way to do this might be to add a prop to open the dropdown, and then set that prop to |
@AbhineetJain Thanks this is great - as are your callouts. Would you be down to add a story for this one? I think it could be the case that our snapshot is always picking up the menu when it's in a closed state. |
We don't snapshot test with jest - we use Storybook & Chromatic. We'd see if any snapshots changed or were broken in CI. Edit: Screenshot |
@Kira-Pilot @vapurrmaid Thanks for the suggestions. Let me add a new story with the dropdown open. |
site/src/components/UserProfileCard/UserProfileCard.stories.tsx
Outdated
Show resolved
Hide resolved
31c9145
to
1caefa5
Compare
site/src/components/UserDropdownContent/UserDropdownContent.stories.tsx
Outdated
Show resolved
Hide resolved
site/src/components/UserDropdownContent/UserDropdownContent.stories.tsx
Outdated
Show resolved
Hide resolved
1caefa5
to
926c6ff
Compare
* add a divider after Account menu item * test: improve Storybook tests * add closed and open userdropdown tests * add default isOpen * extract UserDropdownContent into a single component * remove the isOpen prop * address nit comments * update test name
This PR adds a divider after the Account menu item in the User dropdown.
Subtasks
Fixes #1520
Screenshots
Old
New