UN-3334 [FIX] Fix profile dropdown menu items alignment#1861
UN-3334 [FIX] Fix profile dropdown menu items alignment#1861chandrasekharan-zipstack merged 4 commits intomainfrom
Conversation
…of left-aligned Ant Design 5.13 Button type="text" introduced justify-content: center, overriding the default flex-start alignment in the profile dropdown menu. Add explicit justify-content: flex-start to .logout-button and ensure plugin-rendered menu items are also left-aligned via .ant-dropdown-menu-title-content. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Summary by CodeRabbit
WalkthroughCSS adjustments for the top navigation bar: Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can customize the tone of the review comments and chat replies.Configure the |
Greptile SummaryThis is a CSS-only fix that restores left-alignment in the profile dropdown menu, which regressed when the antd dependency (pinned as Key changes:
Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[antd 5.5.1 resolves to 5.13.2] --> B[ant-btn-text gains justify-content center with higher specificity]
B --> C{Menu item type?}
C --> D[logout-button Button - Profile / Custom Plans / Login]
C --> E[Plugin-rendered label - no logout-button class]
D --> F[Fix: add justify-content flex-start to logout-button]
E --> G[Fix: add display flex and justify-content flex-start to ant-dropdown-menu-title-content]
F --> H[Items render left-aligned]
G --> H
Last reviewed commit: 9bc7ca7 |
Address review feedback: text-align alone won't override justify-content on flex children. Making the wrapper itself a flex container with justify-content: flex-start ensures plugin items using antd Button are also left-aligned. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
@muhammad-ali-e Please resolve the lint errors |
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Frontend Lint Report (Biome)✅ All checks passed! No linting or formatting issues found. |
|



What
justify-content: flex-startto.logout-buttonclass.ant-dropdown-menu-title-contentto cover plugin-rendered menu itemsWhy
^5.5.1range), which introducedjustify-content: centeronButton type="text"with higher specificityHow
justify-content: flex-starton.logout-buttonto override antd'sjustify-content: center.ant-dropdown-menu-title-contentrule withtext-align: left; width: 100%to ensure plugin components that don't use thelogout-buttonclass are also left-alignedCan this PR break any existing features?
justify-content: flex-startis the browser default for flex containers, so this explicitly restores expected behavior that antd 5.13 overrode.Database Migrations
Env Config
Related Issues or PRs
Dependencies Versions
Notes on Testing
Checklist