-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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(module:menu): add support nzPlacement for nz-submenu #7420
Conversation
This preview will be available after the AzureCI is passed. |
Codecov Report
@@ Coverage Diff @@
## master #7420 +/- ##
=======================================
Coverage 91.64% 91.64%
=======================================
Files 487 487
Lines 16043 16045 +2
Branches 2610 2611 +1
=======================================
+ Hits 14702 14704 +2
Misses 1028 1028
Partials 313 313
Continue to review full report at Codecov.
|
I don't think we need to expose the concept of |
#4743 I dont find solution it fix use cdkConnectedOverlayPositions. |
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
) Closes NG-ZORRO#4743 Co-authored-by: Alexander Kharkovey <alexander.kharkovey@nordigy.ru>
Closes #4743
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #4743
What is the new behavior?
Add new option for set position sub-menu
Does this PR introduce a breaking change?
Other information