Skip to content

Commit f9a4350

Browse files
committed
[MNY-336] Portal: Fix spacing issue on menu with link (#8560)
<!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on modifying the sidebar configuration and adjusting styles in the `Sidebar` component for better layout and design. ### Detailed summary - In `apps/portal/src/app/bridge/sidebar.tsx`: - Added an entry for `name: "Guides"` in the sidebar configuration. - In `apps/portal/src/components/others/Sidebar.tsx`: - Updated the `className` of a `div` to include `py-1.5` and `px-3`. - Modified the `className` of a `Link` to remove padding classes `py-1.5` and `px-3`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Style** * Improved sidebar category spacing with enhanced padding distribution * Expanded interactive click target area for sidebar navigation items * Better visual organization of the Guides section <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 27d72e5 commit f9a4350

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

apps/portal/src/app/bridge/sidebar.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ export const sidebar: SideBar = {
1919
},
2020
{ separator: true },
2121
{
22+
name: "Guides",
2223
isCollapsible: false,
2324
links: [
2425
{
@@ -76,7 +77,6 @@ export const sidebar: SideBar = {
7677
name: "Custom Data",
7778
},
7879
],
79-
name: "Guides",
8080
},
8181
{ separator: true },
8282
{

apps/portal/src/components/others/Sidebar.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -203,15 +203,15 @@ function DocSidebarCategory(props: {
203203
"text-muted-foreground hover:text-foreground",
204204
)}
205205
>
206-
<div className="flex gap-2" ref={triggerRef}>
206+
<div className="flex gap-2 py-1.5 px-3" ref={triggerRef}>
207207
{icon && <SidebarIcon icon={icon} />}
208208
{name}
209209
</div>
210210
</div>
211211
);
212212

213213
const triggerEl = href ? (
214-
<Link className={cn("w-full py-1.5 px-3 text-left")} href={href}>
214+
<Link className={cn("w-full text-left")} href={href}>
215215
{triggerElContent}
216216
</Link>
217217
) : (

0 commit comments

Comments
 (0)