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 : Long labels overflowing in extended navigation rail. #145474
🐛 : FIX : Long labels overflowing in extended navigation rail. #145474
Conversation
Closed by mistake, re-opened the PR. |
@aliasgar4558 Sorry for the burden, you will have to rebase this PR on the FYI, for the other Flutter repos, main branch is ok, especially for |
@bleroux - Thanks for the suggestions. It seems good to have those in the PR so will add that. Also, Just to confirm - instead of |
Yes, this is required in flutter/flutter. Sorry for the burden. |
@bleroux - Not problem at all. I am here to learn and contribute & you guys are helping me. So all good. Will do the needful and update PR. |
@bleroux - I have updated test as per your suggested changes & now PR is targeting to |
const String normalLabel = 'Abc'; | ||
const String longLabel = 'Very long bookmark text for navigation destination'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Please move these declarations above (line 3637) and use these variables in the pumpWidget body (lines 3653 & 3658).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Updated now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for your contribution 🙏
@dkwingsmt @bleroux - Thank you so much for your valuable insights for this PR. Really Appreciate it 😊 |
Thank YOU for the contribution, and continuously addressing our comments! |
It has been observed that while in extended navigation rail, if longer labels are given the it is overflowing. Problem identified in "NavigationRail" widget only in case of extended version of it. This PR includes the fix for the same.
Fixes #110901.
NO breaking changes.
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on [Discord].