fix(module:select): icons not showing multi (#5575 #5568)

[DepickereSven]

close #5575 #5568

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #5575 and #5568

What is the new behavior?

Arrow icon and search icon can be used in multi mode

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

[zorro-bot]

This preview will be available after the AzureCI is passed.

[codecov]

Codecov Report

Merging #5650 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5650      +/-   ##
==========================================
+ Coverage   90.04%   90.05%   +0.01%     
==========================================
  Files         455      455              
  Lines       13847    13847              
  Branches     2266     2266              
==========================================
+ Hits        12468    12470       +2     
+ Misses        852      851       -1     
+ Partials      527      526       -1     

Impacted Files	Coverage Δ
components/select/select.component.ts	87.54% <ø> (ø)
components/core/util/text-measure.ts	91.30% <0.00%> (+1.73%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1afabd4...1a2f95a. Read the comment docs.

[DepickereSven]

@vthinkxie

Some screenshots to prove that it works

[vthinkxie]

Hi @DepickereSven

按照 ant design 设计规范，在多选模式下是没有下拉箭头的，在 9.x 之后同步了 antd 4.x 的设计规范，因此多选箭头被移除了，这是在预期中的，不是一个bug

感谢你的 PR

[DepickereSven]

@vthinkxie

By ant-design are you referring to this library https://github.com/ant-design/ant-design/?

Because when I look in their docs I find the following:

So for multi-select and tags is by default on false. But it's possible to set the showArrow on true so you can see it.

Like you can see in the following example the icons work in multi and tag mode.

https://stackblitz.com/edit/react-vrppip?file=index.js

[Hx2heng]

Hi @DepickereSven

按照 ant design 设计规范，在多选模式下是没有下拉箭头的，在 9.x 之后同步了 antd 4.x 的设计规范，因此多选箭头被移除了，这是在预期中的，不是一个bug

感谢你的 PR

在多选模式下，设置nzShowArrow为true没有箭头出现。（ng-zorro 版本9.3.0）

[DepickereSven]

@Hx2heng

This has been solved in the following PR #5785.
This PR is included in version 10.0.0.
So it's normal that it doesn't work in 9.3.0.

So to get this bug solved I would suggest you upgrade to Angular 10. So you can use ng-zorro 10.0.0.
Changelog of version 10.0.0 => https://github.com/NG-ZORRO/ng-zorro-antd/releases/tag/10.0.0

[Hx2heng]

@DepickereSven Thanks.

[zds-d]

In version 10.x, tree-select component already has nzShowArrow property, but nz-tree-select component still does not have this property