-
-
Notifications
You must be signed in to change notification settings - Fork 999
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(module:menu): Wrap IconTemplate in corrected span #2006
Conversation
* wrap IconTemplate using span created by Icon component so InlineCollapse properly displays IconTemplate *docs: Added missing docs for IconTemplate
Codecov Report
@@ Coverage Diff @@
## master #2006 +/- ##
==========================================
+ Coverage 25.38% 27.29% +1.91%
==========================================
Files 488 511 +23
Lines 32098 24481 -7617
Branches 0 233 +233
==========================================
- Hits 8148 6683 -1465
+ Misses 23950 17762 -6188
- Partials 0 36 +36
Continue to review full report at Codecov.
|
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.
Something weird happened with change detection - git should detect 3 new lines, but detects much more. Anyway, LGTM.
* fix(module:menu): Wrap IconTemplate in corrected span * wrap IconTemplate using span created by Icon component so InlineCollapse properly displays IconTemplate *docs: Added missing docs for IconTemplate * Update icontemplate.md Co-authored-by: James Yeung <shunjiey@hotmail.com>
* fix(module:menu): Wrap IconTemplate in corrected span * wrap IconTemplate using span created by Icon component so InlineCollapse properly displays IconTemplate *docs: Added missing docs for IconTemplate * Update icontemplate.md Co-authored-by: James Yeung <shunjiey@hotmail.com>
* fix(module:menu): Wrap IconTemplate in corrected span * wrap IconTemplate using span created by Icon component so InlineCollapse properly displays IconTemplate *docs: Added missing docs for IconTemplate * Update icontemplate.md Co-authored-by: James Yeung <shunjiey@hotmail.com>
* fix(module:menu): Wrap IconTemplate in corrected span * wrap IconTemplate using span created by Icon component so InlineCollapse properly displays IconTemplate *docs: Added missing docs for IconTemplate * Update icontemplate.md Co-authored-by: James Yeung <shunjiey@hotmail.com>
🤔 This is a ...
🔗 Related issue link
fixes #2005
💡 Background and solution
When using the
IconTemplate
in the Menu, collapsing the menu will not properly format the icon and text. WrappingIconTemplate
RenderFragment in the proper span now allows the collapse to properly work without custom code from user.📝 Changelog
☑️ Self Check before Merge