-
-
Notifications
You must be signed in to change notification settings - Fork 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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(module:transfer): transfer button spacing #2156
fix(module:transfer): transfer button spacing #2156
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2156 +/- ##
==========================================
+ Coverage 27.86% 29.67% +1.80%
==========================================
Files 492 519 +27
Lines 32544 25308 -7236
Branches 0 243 +243
==========================================
- Hits 9070 7510 -1560
+ Misses 23474 17762 -5712
- 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.
LGTM
* 'master' of https://github.com/knight1219/ant-design-blazor: Allowing FormItem to be used in default EditForm refactoring EditContext to be injected properly by both AndDesign.Form and EditForm. fix(module: transfer): transfer button spacing (ant-design-blazor#2156) docs: Improve the API part about Form (ant-design-blazor#2150) chore: support IE and .net6 for server-side (ant-design-blazor#2119) docs: Update getting-started.zh-CN.md (ant-design-blazor#2145) docs(module: list): update the demos to use avatar template (ant-design-blazor#2147) fix(module: select): it's incorrect to detect type this way, it throws when class inheritance is used (ant-design-blazor#2121) fix(module: tree): Update check status when set `CheckedKeys` property (ant-design-blazor#2133) fix(module: radio): `Radio` in a group should always keep `Disabled` as same as `RadioGroup.Disabled` (ant-design-blazor#2142)
馃 This is a ...
馃敆 Related issue link
馃挕 Background and solution
The buttons for tranfering items from one side to the other had unnecessary spacing.
This is caused by the
<span>
element which is rendered inside the button.Since the button includes an API to prevent this, I added a check for it.
馃摑 Changelog
NoSpanWrap
to button when string is emty鈽戯笍 Self Check before Merge