-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
feat: globally disable underscore for links in menu items #11491
Conversation
cc @kgabryje |
Codecov Report
@@ Coverage Diff @@
## master #11491 +/- ##
==========================================
- Coverage 66.54% 62.29% -4.25%
==========================================
Files 873 873
Lines 41767 41762 -5
Branches 3826 3826
==========================================
- Hits 27792 26017 -1775
- Misses 13873 15566 +1693
- Partials 102 179 +77
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
6db7ca6
to
ce4f3d6
Compare
ce4f3d6
to
7bcd436
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.
LGTM!
SUMMARY
Globally disable underscore for links in menu items. Adding style overrides with styled components.
Developers should now use
import { MenuItem } from "src/common/components"
, instead ofMenu.Item
.Also makes antd exports explicit so that it's easier to override components under the same name.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before
#11395 (comment)
After
TEST PLAN
Manual
ADDITIONAL INFORMATION