Skip to content

Commit c422a30

Browse files
siddhant1Siddhantclaude
authored
fix(ui): stop marketplace sidebar padding shift at 1023px (#27119)
* fix(ui): stop marketplace sidebar padding shift at 1023px The core NavList uses `tw:px-2 tw:lg:px-4`, causing an 8px horizontal jump as the viewport crosses Tailwind's lg (1024px) breakpoint. Override the padding from the marketplace Sidebar consumer with a constant `tw:px-4` so nav items stay put on resize. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(ui): cap expanded sidebar brand logo at 150px Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Siddhant <siddhant@MacBook-Pro-457.local> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 65effdb commit c422a30

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

openmetadata-ui/src/main/resources/ui/src/components/Sidebar/Sidebar.component.tsx

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ const Sidebar = ({ className }: SidebarProps) => {
9999
/>
100100
) : (
101101
<BrandImage
102-
className="tw:h-10 tw:w-auto"
102+
className="tw:h-10 tw:w-auto tw:max-w-[150px]"
103103
dataTestId="image"
104104
height={40}
105105
width="auto"
@@ -109,14 +109,18 @@ const Sidebar = ({ className }: SidebarProps) => {
109109
</div>
110110

111111
<div className="app-sidebar-link tw:flex tw:flex-col tw:flex-1 tw:overflow-y-auto">
112-
<NavList activeUrl={activeUrl} items={config.items} />
112+
<NavList
113+
activeUrl={activeUrl}
114+
className="tw:px-4"
115+
items={config.items}
116+
/>
113117

114118
{config.bottomItems && (
115119
<>
116120
<div className="tw:flex-1" />
117121
<NavList
118122
activeUrl={activeUrl}
119-
className="tw:pb-4"
123+
className="tw:px-4 tw:pb-4"
120124
items={config.bottomItems}
121125
/>
122126
</>

0 commit comments

Comments
 (0)