feat(module:modal): draggable

[ParsaArvanehPA]

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?

[ ] 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?

Ng-zorro's Modal component lacks the draggable feature that Ant designs' modal provides.

What is the new behavior?

Now supports draggable modals

Does this PR introduce a breaking change?

[ ] Yes
[✔] No

Other information

[zorro-bot]

This preview will be available after the AzureCI is passed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.64%. Comparing base (fdfc816) to head (823a4db).
Report is 4 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #8419      +/-   ##
==========================================
+ Coverage   91.61%   91.64%   +0.02%     
==========================================
  Files         519      519              
  Lines       18076    18122      +46     
  Branches     2848     2769      -79     
==========================================
+ Hits        16561    16607      +46     
- Misses       1216     1217       +1     
+ Partials      299      298       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lppedd]

That's awesome! I was literally thinking today users would benefit from being able to move a modal dialog.
This brings what we can do in desktop environments to web applications. nzMask = false might be important in this case.

[lppedd]

Btw, someone else had attempted at this a while back: #7852

Hopefully this one gets merged.

[lppedd]

@ParsaArvanehPA does it make sense for the automation to still assign @hsuanxyz as reviewer?

Seems like he's not involved anymore.

[ParsaArvanehPA]

That's awesome! I was literally thinking today users would benefit from being able to move a modal dialog. This brings what we can do in desktop environments to web applications. nzMask = false might be important in this case.

Thanks :))
Yeah not to be able to drag modals is really noticible in out modal component.
I saw that old PR, but it it seemed abandoned...

[ParsaArvanehPA]

@ParsaArvanehPA does it make sense for the automation to still assign @hsuanxyz as reviewer?

Seems like he's not involved anymore.

Assigneess are automatically assigned based on this file (I think).

I don't know if they are still involved or not. But other collabrators will check this PR nonetheless.

[HyperLife1119]

Change default to auto.

[ParsaArvanehPA]

@HyperLife1119
Thank you for checking my PR; changes are done.

[HyperLife1119]

LGTM

[lppedd]

Thanks guys for the fast merge, appreciate!