-
Notifications
You must be signed in to change notification settings - Fork 4k
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:transfer): correctly set the transfer-list-body class #8960
Conversation
This preview will be available after the AzureCI is passed. |
@@ -63,10 +63,7 @@ import { NzTransferSearchComponent } from './transfer-search.component'; | |||
<span class="ant-transfer-list-header-title">{{ titleText }}</span> | |||
} | |||
</div> | |||
<div | |||
class="{{ showSearch ? 'ant-transfer-list-body ant-transfer-list-body-with-search' : 'ant-transfer-list-body' }}" | |||
[class]="{ 'ant-transfer__nodata': stat.shownCount === 0 }" |
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.
ant-transfer__nodata
这个 class 不用了吗
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.
antd 中并不存在这个 class,也没有相关样式,应该是遗留的旧名称,我想可以直接去除,WDYT?
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.
可以的
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8960 +/- ##
=======================================
Coverage 91.91% 91.91%
=======================================
Files 555 555
Lines 19708 19708
Branches 2943 2943
=======================================
Hits 18114 18114
Misses 1268 1268
Partials 326 326 ☔ View full report in Codecov by Sentry. |
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
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: N/A
由于 class 未正确设置,导致样式错误。
What is the new behavior?
Does this PR introduce a breaking change?
Other information