Skip to content

chore(angular): generate standalone component wrappers - #27970

Merged
sean-perkins merged 6 commits into
FW-4612from
sp/angular-standalone-proxies
Aug 18, 2023
Merged

chore(angular): generate standalone component wrappers#27970
sean-perkins merged 6 commits into
FW-4612from
sp/angular-standalone-proxies

Conversation

@sean-perkins

@sean-perkins sean-perkins commented Aug 10, 2023

Copy link
Copy Markdown
Contributor

Issue number: N/A


What is the current behavior?

Ionic Framework currently only generates the lazy/hydrated Angular component wrappers for the web components.

What is the new behavior?

  • Generates Angular standalone component wrappers for the web components, using the CE build.
    • Adds manual component wrapper for ion-icon with the CE build.
    • Migrates the ion-back-button and ion-nav to be manual component wrappers
  • Continues to generate the lazy/hydrated Angular component wrappers.
  • Refactors "NavDelegate" etc. language to IonNav to avoid exporting as and simplify navigating the code.

Does this introduce a breaking change?

  • Yes
  • No

Other information

Related output targets PR: stenciljs/output-targets#367

@github-actions github-actions Bot added package: core @ionic/core package package: angular @ionic/angular package labels Aug 10, 2023
@sean-perkins
sean-perkins marked this pull request as ready for review August 14, 2023 17:48
@sean-perkins
sean-perkins requested review from a team, brandyscarney and liamdebeasi and removed request for a team August 14, 2023 17:48
Comment thread core/stencil.config.ts
@sean-perkins
sean-perkins force-pushed the sp/angular-standalone-proxies branch from bed7575 to 8e354d1 Compare August 17, 2023 16:59
@sean-perkins

Copy link
Copy Markdown
Contributor Author

Ok, this is ready for another review. Component wrappers should be generating correctly. IonNav, IonIcon and IonBackButton are all manual component wrappers.

I've updated the test app to render the standalone tests within the generated ion-app standalone component wrapper.

@sean-perkins
sean-perkins merged commit e57759f into FW-4612 Aug 18, 2023
@sean-perkins
sean-perkins deleted the sp/angular-standalone-proxies branch August 18, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: angular @ionic/angular package package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants