🐛 Fix sidebar collapse and pin icons overlapping with sidebar content #3887
🐛 Fix sidebar collapse and pin icons overlapping with sidebar content #3887shubhamkumar9199 wants to merge 1 commit intokubestellar:mainfrom
Conversation
…kubestellar#3886) Signed-off-by: shubhamsharma9199 <shubham.24bcs10320@sst.scaler.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for kubestellarconsole ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
👋 Hey @shubhamkumar9199 — thanks for opening this PR!
This is an automated message. |
📌 Fixes
Fixes #3886
📝 Summary of Changes
sidebarWidth - 14tosidebarWidth + 4so icons render outside the sidebar bordertransition-[left]to keep icons in sync with sidebar collapse/expand animationChanges Made
web/src/components/layout/Sidebar.tsxChecklist
git commit -s)Screenshots or Logs (if applicable)
Before:
After:
👀 Reviewer Notes
The collapse (
<) and pin icons were positioned atsidebarWidth - 14, placing them half-inside the sidebar. Changed tosidebarWidth + 4so they sit just outside the sidebar border. Addedtransition-[left] duration-300to match the sidebar's existing collapse animation.