Skip to content

feat(angular): add standalone modal - #27885

Merged
liamdebeasi merged 20 commits into
FW-4612from
FW-4764
Aug 4, 2023
Merged

feat(angular): add standalone modal#27885
liamdebeasi merged 20 commits into
FW-4612from
FW-4764

Conversation

@liamdebeasi

@liamdebeasi liamdebeasi commented Jul 28, 2023

Copy link
Copy Markdown
Contributor

Issue number: N/A


What is the current behavior?

Modal cannot be used as a standalone component

What is the new behavior?

  • Added modal as a standalone component
  • Added a test
  • Lazy and standalone modal now extend from the same base implementation

Does this introduce a breaking change?

  • Yes
  • No

Other information

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@liamdebeasi liamdebeasi changed the title Fw 4764 feat(angular): add standalone modal Jul 28, 2023
@github-actions github-actions Bot added the package: angular @ionic/angular package label Jul 28, 2023
@@ -0,0 +1,59 @@
// TODO: Is there a way we can grab this from angular-component-lib instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@liamdebeasi
liamdebeasi force-pushed the FW-4764 branch 2 times, most recently from d485de0 to f4a42b4 Compare July 31, 2023 13:58
@liamdebeasi
liamdebeasi marked this pull request as ready for review July 31, 2023 16:04
Base automatically changed from 4612-subdirs to FW-4612 August 1, 2023 19:44

@thetaPC thetaPC left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sean-perkins sean-perkins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar feedback to popover. Implementation changes look good. Just one minor documentation tweak.

Comment thread packages/angular/common/src/overlays/modal.ts Outdated
liamdebeasi and others added 2 commits August 4, 2023 14:48
Co-authored-by: Sean Perkins <sean@ionic.io>
@liamdebeasi
liamdebeasi merged commit 37acdf9 into FW-4612 Aug 4, 2023
@liamdebeasi
liamdebeasi deleted the FW-4764 branch August 4, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: angular @ionic/angular package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants