Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

FIX the color of the menu separator #11564

Merged
merged 11 commits into from
Jul 20, 2020
Merged

FIX the color of the menu separator #11564

merged 11 commits into from
Jul 20, 2020

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Jul 15, 2020

The separator in the menu in dark mode was styled in the light-mode @domyen

Screenshot 2020-07-15 at 15 12 00

@ndelangen ndelangen requested a review from kylesuss July 15, 2020 13:11
@ndelangen ndelangen added this to the 6.0 milestone Jul 15, 2020
@domyen
Copy link
Member

domyen commented Jul 15, 2020

Nice find Norbert!

While you're here, could you break the menu out into its own story?

The tooltip has a startOpen prop (example) that starts the tooltip open so we can see its contents. I think that would help prevent future regressions here especially when it comes to themes.

@domyen
Copy link
Member

domyen commented Jul 16, 2020

The menu:real story doesn't seem to render the menu. Video here.

@ndelangen
Copy link
Member Author

yeah, kinda intentional really :)

Since the other story already covers the expanded state?

@domyen
Copy link
Member

domyen commented Jul 16, 2020

Heads up, this story gets captured strangely in Chromatic. Add more height?

@@ -2,7 +2,7 @@ import React, { useMemo } from 'react';

import { Badge } from '@storybook/components';
import { API } from '@storybook/api';
import { useTheme, Theme, darken } from '@storybook/theming';
import { useTheme, Theme } from '@storybook/theming';
Copy link
Member

Choose a reason for hiding this comment

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

Oops I forgot to remove this sorry. Embarrassing 😅

@ndelangen ndelangen merged commit 661e915 into next Jul 20, 2020
@domyen domyen deleted the fix/color-border-in-menu branch July 20, 2020 11:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants