New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(menu): resolve overflow issue with n-menu root-indent (#5616) #5618
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #5618 +/- ##
==========================================
- Coverage 61.07% 61.07% -0.01%
==========================================
Files 1107 1107
Lines 23619 23621 +2
Branches 6187 6188 +1
==========================================
+ Hits 14426 14427 +1
Misses 7598 7598
- Partials 1595 1596 +1 ☔ View full report in Codecov by Sentry. |
I think it's acceptable to keep current behavior, since if we change it, most menu's appearance would change. So I will revert link PR. |
Also, if someone would like to change overlay's position, a padding starts from edge would it easier to be customized. |
Description of Changes
Adjusted the
paddingLeftRef
calculation logic to ensure proper left margin in different scenarios, addressing the issue of content overflow when setting a smaller root-indent in n-menu.Fixed the calculation of the
::before
pseudo-element's left position, adjusting its default value to match the design expectation and preventing content overflow.Related Issue
Fixes #5616