From 84186e52e997a4dd55f98bf7bc0199656943b9c9 Mon Sep 17 00:00:00 2001 From: Benjamin Canac Date: Fri, 6 Sep 2024 13:00:05 +0200 Subject: [PATCH] fix(NavigationMenu): handle open state hover effect --- src/theme/navigation-menu.ts | 8 +- .../__snapshots__/NavigationMenu.spec.ts.snap | 84 +++++++++---------- 2 files changed, 46 insertions(+), 46 deletions(-) diff --git a/src/theme/navigation-menu.ts b/src/theme/navigation-menu.ts index 617e795f..e68568cf 100644 --- a/src/theme/navigation-menu.ts +++ b/src/theme/navigation-menu.ts @@ -102,8 +102,8 @@ export default (options: Required) => ({ active: false, variant: 'pill', class: { - link: ['hover:text-gray-900 dark:hover:text-white hover:before:bg-gray-50 dark:hover:before:bg-gray-800/50', options.transitions && 'transition-colors before:transition-colors'], - linkLeadingIcon: ['group-hover:text-gray-700 dark:group-hover:text-gray-200', options.transitions && 'transition-colors'] + link: ['hover:text-gray-900 dark:hover:text-white hover:before:bg-gray-50 dark:hover:before:bg-gray-800/50 data-[state=open]:text-gray-900 dark:data-[state=open]:text-white data-[state=open]:before:bg-gray-50 dark:data-[state=open]:before:bg-gray-800/50', options.transitions && 'transition-colors before:transition-colors'], + linkLeadingIcon: ['group-hover:text-gray-700 dark:group-hover:text-gray-200 group-data-[state=open]:text-gray-700 dark:group-data-[state=open]:text-gray-200', options.transitions && 'transition-colors'] } }, ...options.colors.map((color: string) => ({ color, @@ -140,8 +140,8 @@ export default (options: Required) => ({ active: false, variant: 'link', class: { - link: ['hover:text-gray-900 dark:hover:text-white', options.transitions && 'transition-colors'], - linkLeadingIcon: ['group-hover:text-gray-700 dark:group-hover:text-gray-200', options.transitions && 'transition-colors'] + link: ['hover:text-gray-900 dark:hover:text-white data-[state=open]:text-gray-900 dark:data-[state=open]:text-white', options.transitions && 'transition-colors'], + linkLeadingIcon: ['group-hover:text-gray-700 dark:group-hover:text-gray-200 group-data-[state=open]:text-gray-700 dark:group-data-[state=open]:text-gray-200', options.transitions && 'transition-colors'] } }, ...options.colors.map((color: string) => ({ color, diff --git a/test/components/__snapshots__/NavigationMenu.spec.ts.snap b/test/components/__snapshots__/NavigationMenu.spec.ts.snap index 5b72ad20..1f43058e 100644 --- a/test/components/__snapshots__/NavigationMenu.spec.ts.snap +++ b/test/components/__snapshots__/NavigationMenu.spec.ts.snap @@ -4,7 +4,7 @@ exports[`NavigationMenu > renders with arrow correctly 1`] = ` "