Skip to content

Commit 6e61864

Browse files
fix(sidebar): prevent overflow in SidebarSeparator with horizontal orientation
Updated SidebarSeparator component to use `data-[orientation=horizontal]:w-auto` instead of `w-auto` to prevent overflow issues caused by underlying Separator classes. The change makes the width override more specific, only applying to horizontal separators, which prevents conflicts with other Separator styling.
1 parent 2de91c6 commit 6e61864

6 files changed

Lines changed: 9 additions & 9 deletions

File tree

apps/v4/registry/new-york-v4/ui/sidebar/SidebarSeparator.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const props = defineProps<{
1212
<Separator
1313
data-slot="sidebar-separator"
1414
data-sidebar="separator"
15-
:class="cn('bg-sidebar-border mx-2 w-auto', props.class)"
15+
:class="cn('bg-sidebar-border mx-2 data-[orientation=horizontal]:w-auto', props.class)"
1616
>
1717
<slot />
1818
</Separator>

apps/www/src/public/r/styles/default/sidebar.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@
154154
},
155155
{
156156
"path": "ui/sidebar/SidebarSeparator.vue",
157-
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/default/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-sidebar=\"separator\"\n :class=\"cn('mx-2 w-auto bg-sidebar-border', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
157+
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/default/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-sidebar=\"separator\"\n :class=\"cn('mx-2 data-[orientation=horizontal]:w-auto bg-sidebar-border', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
158158
"type": "registry:ui",
159159
"target": ""
160160
},
@@ -213,4 +213,4 @@
213213
"sidebar-ring": "217.2 91.2% 59.8%"
214214
}
215215
}
216-
}
216+
}

apps/www/src/public/r/styles/new-york-v4/sidebar.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@
154154
},
155155
{
156156
"path": "ui/sidebar/SidebarSeparator.vue",
157-
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/new-york/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-slot=\"sidebar-separator\"\n data-sidebar=\"separator\"\n :class=\"cn('bg-sidebar-border mx-2 w-auto', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
157+
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/new-york-v4/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-slot=\"sidebar-separator\"\n data-sidebar=\"separator\"\n :class=\"cn('bg-sidebar-border mx-2 data-[orientation=horizontal]:w-auto', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
158158
"type": "registry:ui",
159159
"target": ""
160160
},
@@ -213,4 +213,4 @@
213213
"sidebar-ring": "217.2 91.2% 59.8%"
214214
}
215215
}
216-
}
216+
}

apps/www/src/public/r/styles/new-york/sidebar.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@
154154
},
155155
{
156156
"path": "ui/sidebar/SidebarSeparator.vue",
157-
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/new-york/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-sidebar=\"separator\"\n :class=\"cn('mx-2 w-auto bg-sidebar-border', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
157+
"content": "<script setup lang=\"ts\">\nimport type { HTMLAttributes } from \"vue\"\nimport { cn } from \"@/lib/utils\"\nimport { Separator } from \"@/registry/new-york/ui/separator\"\n\nconst props = defineProps<{\n class?: HTMLAttributes[\"class\"]\n}>()\n</script>\n\n<template>\n <Separator\n data-sidebar=\"separator\"\n :class=\"cn('mx-2 data-[orientation=horizontal]:w-auto bg-sidebar-border', props.class)\"\n >\n <slot />\n </Separator>\n</template>\n",
158158
"type": "registry:ui",
159159
"target": ""
160160
},
@@ -213,4 +213,4 @@
213213
"sidebar-ring": "217.2 91.2% 59.8%"
214214
}
215215
}
216-
}
216+
}

apps/www/src/registry/default/ui/sidebar/SidebarSeparator.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const props = defineProps<{
1111
<template>
1212
<Separator
1313
data-sidebar="separator"
14-
:class="cn('mx-2 w-auto bg-sidebar-border', props.class)"
14+
:class="cn('mx-2 data-[orientation=horizontal]:w-auto bg-sidebar-border', props.class)"
1515
>
1616
<slot />
1717
</Separator>

apps/www/src/registry/new-york/ui/sidebar/SidebarSeparator.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const props = defineProps<{
1111
<template>
1212
<Separator
1313
data-sidebar="separator"
14-
:class="cn('mx-2 w-auto bg-sidebar-border', props.class)"
14+
:class="cn('mx-2 data-[orientation=horizontal]:w-auto bg-sidebar-border', props.class)"
1515
>
1616
<slot />
1717
</Separator>

0 commit comments

Comments
 (0)