-
Notifications
You must be signed in to change notification settings - Fork 11.4k
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
regression: sidebar scrolling horizontally #35048
Conversation
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release-7.3.0 #35048 +/- ##
==============================================
Coverage 59.17% 59.17%
==============================================
Files 2821 2821
Lines 68131 68131
Branches 15152 15152
==============================================
Hits 40316 40316
Misses 24981 24981
Partials 2834 2834
Flags with carried forward coverage won't be shown. Click here to find out more. |
|
I think there are some reason why I added this class to the menu, please give me some time to look into it, until we get it merged |
@dougfabris I went ahead with because, regardless of the reasoning, it isn't right. Please keep investigating; an edge case can be covered in a patch. |
Proposed changes (including videos or screenshots)
Remove class from the sidebar item menu button, as it is already present on its parent element. The style rules of this class was causing a horizontal scroll on the sidebar of the new navigation.
Issue(s)
Steps to test or reproduce
Further comments