From aebf0b3dca50c51c093cb6abf16c4fd995fc1b39 Mon Sep 17 00:00:00 2001 From: Benjamin Canac Date: Fri, 2 May 2025 11:48:34 +0200 Subject: [PATCH] fix(NavigationMenu): remove `sm:w-auto` from content slot Resolves #3987 Reverts https://github.com/nuxt/ui/commit/abe0859691e06564f68335bd82dcd121e976408e --- .../navigation-menu/NavigationMenuContentSlotExample.vue | 1 + src/theme/navigation-menu.ts | 2 +- test/components/__snapshots__/NavigationMenu-vue.spec.ts.snap | 4 ++-- test/components/__snapshots__/NavigationMenu.spec.ts.snap | 4 ++-- 4 files changed, 6 insertions(+), 5 deletions(-) diff --git a/docs/app/components/content/examples/navigation-menu/NavigationMenuContentSlotExample.vue b/docs/app/components/content/examples/navigation-menu/NavigationMenuContentSlotExample.vue index b5a6cb84..8ec2ec6d 100644 --- a/docs/app/components/content/examples/navigation-menu/NavigationMenuContentSlotExample.vue +++ b/docs/app/components/content/examples/navigation-menu/NavigationMenuContentSlotExample.vue @@ -65,6 +65,7 @@ const items = [ class="w-full justify-center" :ui="{ viewport: 'sm:w-(--reka-navigation-menu-viewport-width)', + content: 'sm:w-auto', childList: 'sm:w-96', childLinkDescription: 'text-balance line-clamp-2' }" diff --git a/src/theme/navigation-menu.ts b/src/theme/navigation-menu.ts index be4dae6f..4b182746 100644 --- a/src/theme/navigation-menu.ts +++ b/src/theme/navigation-menu.ts @@ -27,7 +27,7 @@ export default (options: Required) => ({ separator: 'px-2 h-px bg-border', viewportWrapper: 'absolute top-full left-0 flex w-full', viewport: 'relative overflow-hidden bg-default shadow-lg rounded-md ring ring-default h-(--reka-navigation-menu-viewport-height) w-full transition-[width,height,left] duration-200 origin-[top_center] data-[state=open]:animate-[scale-in_100ms_ease-out] data-[state=closed]:animate-[scale-out_100ms_ease-in] z-[1]', - content: 'absolute top-0 left-0 w-full sm:w-auto', + content: 'absolute top-0 left-0 w-full', indicator: 'absolute data-[state=visible]:animate-[fade-in_100ms_ease-out] data-[state=hidden]:animate-[fade-out_100ms_ease-in] data-[state=hidden]:opacity-0 bottom-0 z-[2] w-(--reka-navigation-menu-indicator-size) translate-x-(--reka-navigation-menu-indicator-position) flex h-2.5 items-end justify-center overflow-hidden transition-[translate,width] duration-200', arrow: 'relative top-[50%] size-2.5 rotate-45 border border-default bg-default z-[1] rounded-xs' }, diff --git a/test/components/__snapshots__/NavigationMenu-vue.spec.ts.snap b/test/components/__snapshots__/NavigationMenu-vue.spec.ts.snap index 8e65ef52..61d14ea3 100644 --- a/test/components/__snapshots__/NavigationMenu-vue.spec.ts.snap +++ b/test/components/__snapshots__/NavigationMenu-vue.spec.ts.snap @@ -1226,7 +1226,7 @@ exports[`NavigationMenu > renders with unmountOnHide correctly 1`] = `