-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Change the behavior of the header menus #37490
Conversation
Codecov Report
@@ Coverage Diff @@
## master #37490 +/- ##
=========================================
Coverage 64.68% 64.68%
Complexity 19336 19336
=========================================
Files 1277 1277
Lines 75535 75535
Branches 1331 1331
=========================================
Hits 48860 48860
Misses 26283 26283
Partials 392 392
Continue to review full report at Codecov.
|
4deffb8
to
294936b
Compare
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.
Thx 👍 (not a technical review :) )
@ChrisEdS I am not sure why you have merged these phpstan commits into your branch. Actually in your local git clone you just do something like:
|
8de296a
to
ea35add
Compare
Cleanup code Add hover effect to the right menu Use RGBA instead of HEX Fix typo Add changelog entry
ea35add
to
90f08e7
Compare
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.
Tested with chrome and works.
Description
Included in this PR:
Motivation and Context
After feedback from various sites after a dynamic adjustment of the width of the left menu, I implemented this here.
How Has This Been Tested?
Screenshots:
Before:
After:
Types of changes
Checklist: