Skip to content

🐛 Fix sidebar collapse and pin icons overlapping with sidebar content #3887

Open
shubhamkumar9199 wants to merge 1 commit intokubestellar:mainfrom
shubhamkumar9199:fix/sidebar-icon-overlap
Open

🐛 Fix sidebar collapse and pin icons overlapping with sidebar content #3887
shubhamkumar9199 wants to merge 1 commit intokubestellar:mainfrom
shubhamkumar9199:fix/sidebar-icon-overlap

Conversation

@shubhamkumar9199
Copy link
Copy Markdown

📌 Fixes

Fixes #3886

📝 Summary of Changes

  • Fixed collapse chevron and pin icons overlapping with sidebar content
  • Adjusted button container positioning from sidebarWidth - 14 to sidebarWidth + 4 so icons render outside the sidebar border
  • Added smooth transition-[left] to keep icons in sync with sidebar collapse/expand animation

Changes Made

  • Fixed icon positioning in web/src/components/layout/Sidebar.tsx

Checklist

  • I used a coding agent (Claude Code) to generate/review this code
  • I have reviewed the project's contribution guidelines
  • New cards target console-marketplace, not this repo
  • isDemoData is wired correctly (cards show Demo badge when using demo data)
  • I have tested the changes locally and ensured they work as expected
  • All commits are signed with DCO (git commit -s)

Screenshots or Logs (if applicable)

Before:

image

After:

image

👀 Reviewer Notes

The collapse (<) and pin icons were positioned at sidebarWidth - 14, placing them half-inside the sidebar. Changed to sidebarWidth + 4 so they sit just outside the sidebar border. Added transition-[left] duration-300 to match the sidebar's existing collapse animation.

…kubestellar#3886)

Signed-off-by: shubhamsharma9199 <shubham.24bcs10320@sst.scaler.com>
@kubestellar-prow kubestellar-prow bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Mar 30, 2026
@kubestellar-prow
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for kubestellarconsole ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9ddfe1d
🔍 Latest deploy log https://app.netlify.com/projects/kubestellarconsole/deploys/69c9bd2f06b36f00089b35be
😎 Deploy Preview https://deploy-preview-3887.console-deploy-preview.kubestellar.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kubestellar-prow kubestellar-prow bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

👋 Hey @shubhamkumar9199 — thanks for opening this PR!

🤖 This project is developed exclusively using AI coding assistants.

Please do not attempt to code anything for this project manually.
All contributions should be authored using an AI coding tool such as:

This ensures consistency in code style, architecture patterns, test coverage,
and commit quality across the entire codebase.


This is an automated message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: 🐛 Sidebar collapse and pin icons overlap with sidebar content

1 participant