Skip to content

Fix: Center ThunderLogoSVG and align layout - #63

Open
PavanKuppili wants to merge 1 commit into
Muneerali199:mainfrom
PavanKuppili:fix/thunder-side-navbar-alignment
Open

Fix: Center ThunderLogoSVG and align layout#63
PavanKuppili wants to merge 1 commit into
Muneerali199:mainfrom
PavanKuppili:fix/thunder-side-navbar-alignment

Conversation

@PavanKuppili

Copy link
Copy Markdown

🔧 Pull Request: Fix Sidebar Logo and other components Alignment in Thunder UI

Overview:
This PR addresses a subtle but noticeable misalignment in the Thunder sidebar logo. The SVG was visually offset to the right, causing inconsistency with adjacent navigation items. The fix ensures pixel-perfect centering and smooth layout behavior across collapsed and expanded sidebar states.

Changes Made:

  • Applied a translateX(-2px) transform to the ThunderLogoSVG to correct horizontal drift.
  • Wrapped the logo in a fixed-size container (w-8 h-8) with flex alignment for consistent positioning.
  • Verified layout stability during hover and sidebar toggle transitions.
  • Ensured no impact on other nav components or overall responsiveness.
    **** make use of prettier for document organigation thatsy it seems every thing is changes but changes made in thunder componrnt of line 28 and nav bar div of line from 268 to 398**
    Why It Matters:
    Precise alignment reinforces visual polish and brand consistency. This fix eliminates layout jitter and ensures the Thunder logo appears intentional and balanced within the sidebar.

issue closes: #59

check out the below video clip and observe changes:-
https://github.com/user-attachments/assets/f4834a14-cdf9-4f91-8d06-12441606f469

and is an assigned issue only not a un assigned issue

@vercel

vercel Bot commented Aug 30, 2025

Copy link
Copy Markdown

@PavanKuppili is attempting to deploy a commit to the muneerali199's projects Team on Vercel.

A member of the Team first needs to authorize it.

@PavanKuppili

Copy link
Copy Markdown
Author

hey @Muneerali199 i have done my job and requesting foe review and pull request and let me know if any changes needed. If this project maintains contributes list i ll be honored to be part of then.
thankyou.

@Muneerali199

Copy link
Copy Markdown
Owner

@PavanKuppili please show me screenshot

@PavanKuppili

PavanKuppili commented Aug 31, 2025

Copy link
Copy Markdown
Author

Thankyou for replying my bad sorry for the inconvenience i have added a screen record in the pull request itself
![Untitled video - Made with Clipchamp (1) (1)](https://github.com/user-attachments/assets/09089c3d
-4514-4a86-a852-225279bf35a6)

Screenshot 2025-08-31 091951

@Muneerali199 please kindly check it and me know if any changes needed.
Thank You.

@PavanKuppili

Copy link
Copy Markdown
Author

hey @Muneerali199 i have shared the screen shot and a video clip regarding to the changes i made please kindly review them and let me know if any changes needed.

just a basic remainder. Apologies for any inconvenience.
Thankyou.

@PavanKuppili

Copy link
Copy Markdown
Author

hey @Muneerali199 it is just a casual remainder regarding to the pull requist i made can u please check it out as soon as possible and add the image regarding to the issue i resolved and also a video clip is also added regarding to the hove of that side nav bar after effect.
thank you.

@PavanKuppili PavanKuppili mentioned this pull request Sep 9, 2025
@PavanKuppili

Copy link
Copy Markdown
Author

hey @Muneerali199 hope that this msg finds u well. i have pulled a request for review 2 weeks ago regarding to the changes i made and it is unnoticed till now . hope that every thing going well and let me know if any changes are needed and check the pr as soon as possible.
thankyou.

@PavanKuppili

Copy link
Copy Markdown
Author

hey @Muneerali199 i hope this msg find u well. i created this pull request a month ago and is unnoticed can u check it out and let me know if any changes are needed.
i have attached the images and some video files regarding the visual changes i made refer them for the changes observation.
thankyou.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI enhancement

3 participants