[draft] fix(module:collapse): fix closed collapse cannot be reopned in some …

[wzhudev]

…browsers

close #3098

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) Need Safari or IE or Edge to test it.
• [ ] Docs have been added / updated (for bug fixes / features) Not needed.

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: #3098

In browsers like Safari, IE or Edge, closed panels cannot be reopened because display: none style is not removed.

What is the new behavior?

Use CSS visibility property instead of display. Now on Safari, collapsed panels can be un-collapsed.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

[netlify]

Deploy preview for ng-zorro-master ready!

Built with commit 3d01b9d

https://deploy-preview-3216--ng-zorro-master.netlify.com

[codecov]

Codecov Report

Merging #3216 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3216   +/-   ##
=======================================
  Coverage   97.37%   97.37%           
=======================================
  Files         559      559           
  Lines       11627    11627           
  Branches      835      835           
=======================================
  Hits        11322    11322           
  Misses        193      193           
  Partials      112      112

Impacted Files	Coverage Δ
components/core/animation/collapse.ts	100% <ø> (ø)	⬆️

---

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 51c0d53...3d01b9d. Read the comment docs.

[wzhudev]

I am going to dig deeper on this potential bug of Angular.

[RayXia95]

I have the same exact problem with unable to expand nz-collapse across other desktop browsers and mobile browsers. This fix should be tested on mobile browsers also.

[RayXia95]

Why was this PR closed?

[kjavia]

Anyone facing this issue can switch off animations (as a workaround, until this is officially fixed)

https://deploy-preview-3216--ng-zorro-master.netlify.com/docs/animations/en

[wzhudev]

@RayXia95 Because it's a bug of Angular's and this fix is not neat. But you can use this as a hack.

[KerryXuanWang]

Anyone facing this issue can switch off animations (as a workaround, until this is officially fixed)

https://deploy-preview-3216--ng-zorro-master.netlify.com/docs/animations/en

can't open it now, can you please give an available link?