Skip to content

Commit

Permalink
Fix data checked state in generic menu (#174)
Browse files Browse the repository at this point in the history
  • Loading branch information
vineethasok authored Oct 11, 2023
1 parent e038615 commit 42b8bc1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/components/Dropdown/Dropdown.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ const DropdownExample = ({ showArrow, disabled, side, ...props }: Props) => {
side={side}
>
<Dropdown.Group>
<Dropdown.Item>Content0</Dropdown.Item>
<Dropdown.Item data-state="checked">Content0</Dropdown.Item>
</Dropdown.Group>
<Dropdown.Item icon="activity">Content1 long text content</Dropdown.Item>
<Dropdown.Sub>
Expand Down
2 changes: 1 addition & 1 deletion src/components/GenericMenu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ export const GenericMenuItem = styled.div`
color:${theme.click.genericMenu.item.color.text.hover};
cursor: pointer;
}
&[data-state="open"], &[data-state-"checked"] {
&[data-state="open"], &[data-state="checked"], &[data-selected="true"] {
background:${theme.click.genericMenu.item.color.background.active};
color:${theme.click.genericMenu.item.color.text.active};
font: ${theme.click.genericMenu.item.typography.label.active};
Expand Down

1 comment on commit 42b8bc1

@vercel
Copy link

@vercel vercel bot commented on 42b8bc1 Oct 11, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

click-ui – ./

click-ui-git-main-clickhouse.vercel.app
click-ui-clickhouse.vercel.app
click-ui.vercel.app

Please sign in to comment.