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

build(deps): bump angular to 15 #248

Closed
wants to merge 1 commit into from
Closed

build(deps): bump angular to 15 #248

wants to merge 1 commit into from

Conversation

alan-bio
Copy link

@alan-bio alan-bio commented Mar 9, 2023

Motivation

This library isn't working for Angular 15. Styling is messed up and causes some runtime errors.
Related to #244.

Reason for using Legacy Material instead of the latest MDC update

This is up to the author of the application, but I feel as a first update step, v14.0.0 should support just the legacy material components, and then v16.0.0 can drop the legacy support for the latest MDC components.

Summary of changes

Updated Angular/Material and related dependencies to 15. Using legacy material components for now to ease the upgrade process. This required changing the imports to the Legacy Material modules, removing the appearance="legacy" attribute from mat-form-field and specifying legacy styling in the demo theme.

Angular 15 now requires a es target of 2022 in tsconfig.json. This caused a few runtime errors where class variable initialisers were trying to reference constructor dependencies when they weren't defined yet.

Tests ain't running yet. Final problem to fix.

Updated Angular/Material and related dependencies to 15.
Using legacy material components for now to ease the upgrade process.
This required changing the imports to the Legacy Material modules,
removing the `appearance="legacy"` attribute from `mat-form-field` and
specifying legacy styling in the demo theme.

Angular 15 now requires a es target of `2022` in tsconfig.json.

Tests ain't running yet. Final problem to fix.
@alan-bio alan-bio mentioned this pull request May 10, 2023
@alan-bio alan-bio closed this by deleting the head repository Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants