Skip to content
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

refactor(module:modal): refactor the component #4702

Merged
merged 8 commits into from
Mar 15, 2020

Conversation

hsuanxyz
Copy link
Member

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@codecov
Copy link

codecov bot commented Jan 19, 2020

Codecov Report

Merging #4702 into master will decrease coverage by 0.22%.
The diff coverage is 93.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4702      +/-   ##
==========================================
- Coverage   93.37%   93.15%   -0.23%     
==========================================
  Files         499      513      +14     
  Lines       12753    12939     +186     
  Branches     1965     1971       +6     
==========================================
+ Hits        11908    12053     +145     
- Misses        434      466      +32     
- Partials      411      420       +9
Impacted Files Coverage Δ
components/modal/modal-config.ts 100% <ø> (ø)
components/modal/modal-footer.directive.ts 100% <100%> (ø)
components/modal/modal-close.component.ts 100% <100%> (ø)
components/modal/modal-container.component.ts 100% <100%> (ø)
components/modal/modal-types.ts 100% <100%> (ø)
components/modal/modal-footer.component.ts 100% <100%> (ø)
components/modal/modal-title.component.ts 100% <100%> (ø)
components/modal/modal-animations.ts 100% <100%> (ø)
...ponents/modal/modal-confirm-container.component.ts 100% <100%> (ø)
components/modal/modal.module.ts 100% <100%> (ø)
... and 53 more

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 9f951f8...8c705a2. Read the comment docs.

@zorro-bot
Copy link

zorro-bot bot commented Jan 19, 2020

This preview will be available after the AzureCI is passed.

@hsuanxyz hsuanxyz force-pushed the refactor/modal branch 2 times, most recently from 1f5e9a0 to 4f63fd0 Compare January 20, 2020 08:13
@tieppt
Copy link

tieppt commented Jan 22, 2020

@hsuanxyz Do you consider to implement this feature #4710

@hsuanxyz
Copy link
Member Author

hsuanxyz commented Feb 3, 2020

@hsuanxyz Do you consider to implement this feature #4710

Thank your feedback, we will implement this.

close NG-ZORRO#4710

test(module:modal): add footer directive test

test(module:modal): add callback test

test(module:modal): add animation test

test(module:modal): add classname and styles test

test(module:modal): add footer component test

test(module:modal): fix test

test(module:modal): fix test

test(module:modal): add confirm test

test(module:modal): fix test

test(module:modal): add component mode test

chore: remove console

test: fix ci test
Ricbet pushed a commit to Ricbet/ng-zorro-antd that referenced this pull request Apr 9, 2020
* refactor(module:modal): refactor the modal service

close NG-ZORRO#3652, close NG-ZORRO#4437, close NG-ZORRO#2749

* feat(module:modal): support autofocus

close NG-ZORRO#3423, close NG-ZORRO#4126

* refactor(module:modal): refactor the modal component

* chore: rename files

* test(module:modal): add test case

* test(module:modal): add test case

* feat(module:modal): support nzCloseOnNavigation

* feat(module:modal): support nzViewContainerRef

close NG-ZORRO#4710

test(module:modal): add footer directive test

test(module:modal): add callback test

test(module:modal): add animation test

test(module:modal): add classname and styles test

test(module:modal): add footer component test

test(module:modal): fix test

test(module:modal): fix test

test(module:modal): add confirm test

test(module:modal): fix test

test(module:modal): add component mode test

chore: remove console

test: fix ci test
hsuanxyz added a commit to hsuanxyz/ng-zorro-antd that referenced this pull request Aug 5, 2020
* refactor(module:modal): refactor the modal service

close NG-ZORRO#3652, close NG-ZORRO#4437, close NG-ZORRO#2749

* feat(module:modal): support autofocus

close NG-ZORRO#3423, close NG-ZORRO#4126

* refactor(module:modal): refactor the modal component

* chore: rename files

* test(module:modal): add test case

* test(module:modal): add test case

* feat(module:modal): support nzCloseOnNavigation

* feat(module:modal): support nzViewContainerRef

close NG-ZORRO#4710

test(module:modal): add footer directive test

test(module:modal): add callback test

test(module:modal): add animation test

test(module:modal): add classname and styles test

test(module:modal): add footer component test

test(module:modal): fix test

test(module:modal): fix test

test(module:modal): add confirm test

test(module:modal): fix test

test(module:modal): add component mode test

chore: remove console

test: fix ci test
@hsuanxyz hsuanxyz deleted the refactor/modal branch August 5, 2020 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants