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.

Sign up for GitHub

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

fix(OverflowMenu): use v10 menu and export v11 menu as separate unstable component #10674

Merged
merged 16 commits into from
Feb 23, 2022
Merged

fix(OverflowMenu): use v10 menu and export v11 menu as separate unstable component #10674

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
06f0cde
fix: export v10 overflow menu
jnm2377 Feb 9, 2022
f1c08f4
fix: export v2 overflow menu
jnm2377 Feb 9, 2022
73d57a5
fix: update package index.js exports
jnm2377 Feb 9, 2022
0b9b971
fix: v11 unstable menu inline notifications
jnm2377 Feb 9, 2022
2299f41
fix: v2 overflow menu name
jnm2377 Feb 9, 2022
4b02e5e
fix: v2 overflow menu export
jnm2377 Feb 9, 2022
e4bbf21
fix: v1 overflow menu index.js
jnm2377 Feb 9, 2022
63abde3
chore: update snapshots
jnm2377 Feb 9, 2022
18ebe58
Merge branch 'main' of github.com:carbon-design-system/carbon into v1…
jnm2377 Feb 23, 2022
8fa85f8
fix: v11 breadcrumb overflow menu caret
jnm2377 Feb 23, 2022
e4aa37e
fix: remove class wrapper in menuitem for keyboard
jnm2377 Feb 23, 2022
bd4d02a
fix: add classname prop to v2 menu
jnm2377 Feb 23, 2022
16bd33f
chore: basic v2 overflow menu tests
jnm2377 Feb 23, 2022
b300efe
fix: v11 feature flag stuff
jnm2377 Feb 23, 2022
9a234dd
chore: update snapshots
jnm2377 Feb 23, 2022
ff4ea8b
Merge branch 'main' into v11-overflow-menu-fix
Feb 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: v11 unstable menu inline notifications
  • Loading branch information
@jnm2377
jnm2377 committed Feb 9, 2022
commit 0b9b97149a5249a4b7025f13be5358ab9d03245f
14 changes: 7 additions & 7 deletions packages/react/src/components/ContextMenu/ContextMenu-story.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,13 @@ const Story = (items) => {

return (
<StoryFrame>
<InlineNotification
kind="info"
title="Context menu"
subtitle="Right-click anywhere on this page to access an example implementation of this component."
lowContrast
hideCloseButton
/>
<InlineNotification kind="info" lowContrast hideCloseButton>
<strong>Context Menu</strong>
<p>
Right-click anywhere on this page to access an example implementation
of this component.
</p>
</InlineNotification>
<Menu {...menuProps}>{renderedItems}</Menu>
</StoryFrame>
);
Expand Down
14 changes: 7 additions & 7 deletions packages/react/src/components/Menu/_storybook-utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,13 @@ import {
} from '../Menu';

const InfoBanner = () => (
<InlineNotification
kind="info"
title="Experimental component"
subtitle="This component is considered experimental. Its API may change until the stable version is released."
lowContrast
hideCloseButton
/>
<InlineNotification kind="info" lowContrast hideCloseButton>
<strong>Exerimental component</strong>
<p>
This component is considered experimental. Its API may change until the
stable version is released.
</p>
</InlineNotification>
);

// eslint-disable-next-line react/prop-types
Expand Down