Skip to content

C
coopwire-hypothesis
Commit 3008845d authored by Alejandro Celaya's avatar Alejandro Celaya Committed by Alejandro Celaya
Browse files
Options

Remove pointer and hover on disabled menu items

parent b4eac256
Show whitespace changes
Inline Side-by-side
Showing with 6 additions and 3 deletions
src/sidebar/components/MenuItem.tsx
View file @ 3008845d
...@@ -239,10 +239,11 @@ export default function MenuItem({ ...@@ -239,10 +239,11 @@ export default function MenuItem({
const wrapperClasses = classnames( const wrapperClasses = classnames(
'focus-visible-ring ring-inset', 'focus-visible-ring ring-inset',
'w-full min-w-[150px] flex items-center select-none', 'w-full min-w-[150px] flex items-center select-none',
'rounded-none cursor-pointer', 'rounded-none',
{ {
'focus-visible:rounded-lg': !isSelected, 'focus-visible:rounded-lg': !isSelected,
'focus-visible:rounded-r-lg': isSelected, 'focus-visible:rounded-r-lg': isSelected,
'cursor-pointer': !isDisabled,
}, },
// Set this container as a "group" so that children may style based on its // Set this container as a "group" so that children may style based on its
// layout state. // layout state.
...@@ -251,8 +252,10 @@ export default function MenuItem({ ...@@ -251,8 +252,10 @@ export default function MenuItem({
{ {
'min-h-[30px] font-normal': isSubmenuItem, 'min-h-[30px] font-normal': isSubmenuItem,
'min-h-[40px] font-medium': !isSubmenuItem, 'min-h-[40px] font-medium': !isSubmenuItem,
'bg-grey-1 hover:bg-grey-3': isSubmenuItem || isExpanded, 'bg-grey-1': isSubmenuItem || isExpanded,
'bg-white hover:bg-grey-1': !isSubmenuItem && !isExpanded, 'hover:bg-grey-3': !isDisabled && (isSubmenuItem || isExpanded),
'bg-white': !isSubmenuItem && !isExpanded,
'hover:bg-grey-1': !isDisabled && !isSubmenuItem && !isExpanded,
// visual "padding" on the right is part of SubmenuToggle when rendered, // visual "padding" on the right is part of SubmenuToggle when rendered,
// but when not rendering a SubmenuToggle, we need to add some padding here // but when not rendering a SubmenuToggle, we need to add some padding here
'pr-1': !hasSubmenuVisible, 'pr-1': !hasSubmenuVisible,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment