Skip to content

Commit 8dcec64

Browse files
JamesHollyerJamesHollyer
committed
Angular Component Migration
1 parent 42545de commit 8dcec64

File tree

54 files changed

+292
-296
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+292
-296
lines changed

‎tensorboard/webapp/alert/views/alert_display_snackbar_container.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,7 @@ See the License for the specific language governing permissions and
1313
limitations under the License.
1414
==============================================================================*/
1515
import {ChangeDetectionStrategy, Component, Inject} from '@angular/core';
16-
import {
17-
MatLegacySnackBarRef,
18-
MAT_LEGACY_SNACK_BAR_DATA,
19-
} from '@angular/material/legacy-snack-bar';
16+
import {MatSnackBarRef, MAT_SNACK_BAR_DATA} from '@angular/material/snack-bar';
2017
import {Store} from '@ngrx/store';
2118
import {State} from '../../app_state';
2219
import {splitByURL} from '../../util/string';
@@ -33,8 +30,8 @@ export class AlertDisplaySnackbarContainer {
3330
readonly splitByURL = splitByURL;
3431

3532
constructor(
36-
private readonly snackBarRef: MatLegacySnackBarRef<AlertDisplaySnackbarContainer>,
37-
@Inject(MAT_LEGACY_SNACK_BAR_DATA) readonly unknownData: unknown,
33+
private readonly snackBarRef: MatSnackBarRef<AlertDisplaySnackbarContainer>,
34+
@Inject(MAT_SNACK_BAR_DATA) readonly unknownData: unknown,
3835
private readonly store: Store<State>
3936
) {
4037
this.alert = unknownData as AlertInfo;

‎tensorboard/webapp/alert/views/alert_snackbar_container.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import {
1818
OnDestroy,
1919
OnInit,
2020
} from '@angular/core';
21-
import {MatLegacySnackBar} from '@angular/material/legacy-snack-bar';
21+
import {MatSnackBar} from '@angular/material/snack-bar';
2222
import {Store} from '@ngrx/store';
2323
import {Subject} from 'rxjs';
2424
import {filter, takeUntil} from 'rxjs/operators';
@@ -40,7 +40,7 @@ export class AlertSnackbarContainer implements OnInit, OnDestroy {
4040

4141
constructor(
4242
private readonly store: Store<State>,
43-
private readonly snackBar: MatLegacySnackBar
43+
private readonly snackBar: MatSnackBar
4444
) {}
4545

4646
ngOnInit() {

‎tensorboard/webapp/alert/views/alert_snackbar_module.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ limitations under the License.
1414
==============================================================================*/
1515
import {CommonModule} from '@angular/common';
1616
import {NgModule} from '@angular/core';
17-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
18-
import {MatLegacySnackBarModule} from '@angular/material/legacy-snack-bar';
17+
import {MatButtonModule} from '@angular/material/button';
18+
import {MatSnackBarModule} from '@angular/material/snack-bar';
1919
import {AlertDisplaySnackbarContainer} from './alert_display_snackbar_container';
2020
import {AlertSnackbarContainer} from './alert_snackbar_container';
2121

@@ -25,7 +25,7 @@ import {AlertSnackbarContainer} from './alert_snackbar_container';
2525
@NgModule({
2626
declarations: [AlertSnackbarContainer, AlertDisplaySnackbarContainer],
2727
exports: [AlertSnackbarContainer],
28-
imports: [CommonModule, MatLegacyButtonModule, MatLegacySnackBarModule],
28+
imports: [CommonModule, MatButtonModule, MatSnackBarModule],
2929
entryComponents: [
3030
// Required for non-Ivy Angular apps.
3131
AlertDisplaySnackbarContainer,

‎tensorboard/webapp/alert/views/alert_snackbar_test.ts

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,8 @@ limitations under the License.
1414
==============================================================================*/
1515
import {OverlayContainer} from '@angular/cdk/overlay';
1616
import {TestBed} from '@angular/core/testing';
17-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
18-
import {
19-
MatLegacySnackBar,
20-
MatLegacySnackBarModule,
21-
} from '@angular/material/legacy-snack-bar';
17+
import {MatButtonModule} from '@angular/material/button';
18+
import {MatSnackBar, MatSnackBarModule} from '@angular/material/snack-bar';
2219
import {NoopAnimationsModule} from '@angular/platform-browser/animations';
2320
import {Action, createAction, props, Store} from '@ngrx/store';
2421
import {MockStore, provideMockStore} from '@ngrx/store/testing';
@@ -45,15 +42,11 @@ describe('alert snackbar', () => {
4542
let snackBarOpenSpy: jasmine.Spy;
4643
let recordedActions: Action[] = [];
4744
let overlayContainer: OverlayContainer;
48-
let snackbar: MatLegacySnackBar;
45+
let snackbar: MatSnackBar;
4946

5047
beforeEach(async () => {
5148
await TestBed.configureTestingModule({
52-
imports: [
53-
NoopAnimationsModule,
54-
MatLegacyButtonModule,
55-
MatLegacySnackBarModule,
56-
],
49+
imports: [NoopAnimationsModule, MatButtonModule, MatSnackBarModule],
5750
providers: [
5851
provideMockStore({
5952
initialState: buildStateFromAlertState(buildAlertState({})),
@@ -67,7 +60,7 @@ describe('alert snackbar', () => {
6760
recordedActions.push(action);
6861
});
6962
overlayContainer = TestBed.inject(OverlayContainer);
70-
snackbar = TestBed.inject(MatLegacySnackBar);
63+
snackbar = TestBed.inject(MatSnackBar);
7164
snackBarOpenSpy = spyOn(snackbar, 'openFromComponent').and.callThrough();
7265
});
7366

‎tensorboard/webapp/core/views/layout_module.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ limitations under the License.
1414
==============================================================================*/
1515
import {CommonModule} from '@angular/common';
1616
import {NgModule} from '@angular/core';
17-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
17+
import {MatButtonModule} from '@angular/material/button';
1818
import {MatIconModule} from '@angular/material/icon';
1919
import {LayoutContainer} from './layout_container';
2020

@@ -24,6 +24,6 @@ import {LayoutContainer} from './layout_container';
2424
@NgModule({
2525
declarations: [LayoutContainer],
2626
exports: [LayoutContainer],
27-
imports: [CommonModule, MatIconModule, MatLegacyButtonModule],
27+
imports: [CommonModule, MatIconModule, MatButtonModule],
2828
})
2929
export class LayoutModule {}

‎tensorboard/webapp/feature_flag/views/feature_flag_dialog_module.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ limitations under the License.
1414
==============================================================================*/
1515
import {CommonModule} from '@angular/common';
1616
import {NgModule} from '@angular/core';
17-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
18-
import {MatLegacySelectModule} from '@angular/material/legacy-select';
17+
import {MatButtonModule} from '@angular/material/button';
18+
import {MatSelectModule} from '@angular/material/select';
1919
import {FeatureFlagDialogComponent} from './feature_flag_dialog_component';
2020
import {FeatureFlagDialogContainer} from './feature_flag_dialog_container';
2121

@@ -24,7 +24,7 @@ import {FeatureFlagDialogContainer} from './feature_flag_dialog_container';
2424
*/
2525
@NgModule({
2626
declarations: [FeatureFlagDialogComponent, FeatureFlagDialogContainer],
27-
imports: [CommonModule, MatLegacyButtonModule, MatLegacySelectModule],
27+
imports: [CommonModule, MatButtonModule, MatSelectModule],
2828
exports: [FeatureFlagDialogContainer],
2929
entryComponents: [FeatureFlagDialogContainer],
3030
})

‎tensorboard/webapp/feature_flag/views/feature_flag_modal_trigger_container.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,7 @@ limitations under the License.
1414
==============================================================================*/
1515
import {ComponentType} from '@angular/cdk/overlay';
1616
import {Component, OnDestroy, OnInit} from '@angular/core';
17-
import {
18-
MatLegacyDialog,
19-
MatLegacyDialogRef,
20-
} from '@angular/material/legacy-dialog';
17+
import {MatDialog, MatDialogRef} from '@angular/material/dialog';
2118
import {Store} from '@ngrx/store';
2219
import {Subject} from 'rxjs';
2320
import {takeUntil} from 'rxjs/operators';
@@ -42,12 +39,12 @@ export class FeatureFlagModalTriggerContainer implements OnInit, OnDestroy {
4239
featureFlagDialogType: ComponentType<any> = FeatureFlagDialogContainer;
4340

4441
readonly showFeatureFlags$ = this.store.select(getShowFlagsEnabled);
45-
private featureFlagsDialog?: MatLegacyDialogRef<FeatureFlagDialogContainer>;
42+
private featureFlagsDialog?: MatDialogRef<FeatureFlagDialogContainer>;
4643
private ngUnsubscribe = new Subject<void>();
4744

4845
constructor(
4946
private readonly store: Store<State>,
50-
private dialog: MatLegacyDialog
47+
private dialog: MatDialog
5148
) {}
5249

5350
ngOnInit() {

‎tensorboard/webapp/feature_flag/views/feature_flag_modal_trigger_container_test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ import {HarnessLoader} from '@angular/cdk/testing';
1616
import {TestbedHarnessEnvironment} from '@angular/cdk/testing/testbed';
1717
import {Component} from '@angular/core';
1818
import {ComponentFixture, TestBed} from '@angular/core/testing';
19-
import {MatLegacyDialogModule} from '@angular/material/legacy-dialog';
20-
import {MatLegacyDialogHarness} from '@angular/material/legacy-dialog/testing';
19+
import {MatDialogModule} from '@angular/material/dialog';
20+
import {MatDialogHarness} from '@angular/material/dialog/testing';
2121
import {NoopAnimationsModule} from '@angular/platform-browser/animations';
2222
import {Store} from '@ngrx/store';
2323
import {MockStore} from '@ngrx/store/testing';
@@ -47,7 +47,7 @@ describe('feature_flag_modal_trigger_container', () => {
4747

4848
beforeEach(async () => {
4949
await TestBed.configureTestingModule({
50-
imports: [MatLegacyDialogModule, NoopAnimationsModule],
50+
imports: [MatDialogModule, NoopAnimationsModule],
5151
providers: [provideMockTbStore()],
5252
}).compileComponents();
5353

@@ -78,7 +78,7 @@ describe('feature_flag_modal_trigger_container', () => {
7878
store.overrideSelector(getOverriddenFeatureFlags, {});
7979
store.overrideSelector(getShowFlagsEnabled, true);
8080
createComponent();
81-
const dialog = await rootLoader.getHarness(MatLegacyDialogHarness);
81+
const dialog = await rootLoader.getHarness(MatDialogHarness);
8282
expect(
8383
(fixture.componentInstance as any).featureFlagsDialog
8484
).not.toBeUndefined();

‎tensorboard/webapp/header/dark_mode_toggle_test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ limitations under the License.
1515
import {OverlayContainer} from '@angular/cdk/overlay';
1616
import {NO_ERRORS_SCHEMA} from '@angular/core';
1717
import {ComponentFixture, TestBed} from '@angular/core/testing';
18-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
19-
import {MatLegacyMenuModule} from '@angular/material/legacy-menu';
18+
import {MatButtonModule} from '@angular/material/button';
19+
import {MatMenuModule} from '@angular/material/menu';
2020
import {By} from '@angular/platform-browser';
2121
import {NoopAnimationsModule} from '@angular/platform-browser/animations';
2222
import {Action, Store} from '@ngrx/store';
@@ -37,9 +37,9 @@ describe('dark mode toggle test', () => {
3737
beforeEach(async () => {
3838
await TestBed.configureTestingModule({
3939
imports: [
40-
MatLegacyButtonModule,
40+
MatButtonModule,
4141
MatIconTestingModule,
42-
MatLegacyMenuModule,
42+
MatMenuModule,
4343
NoopAnimationsModule,
4444
],
4545
providers: [provideMockTbStore()],

‎tensorboard/webapp/header/header_module.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ limitations under the License.
1515
// Uses `async` pipe.
1616
import {CommonModule} from '@angular/common';
1717
import {NgModule} from '@angular/core';
18-
import {MatLegacyButtonModule} from '@angular/material/legacy-button';
18+
import {MatButtonModule} from '@angular/material/button';
1919
import {MatIconModule} from '@angular/material/icon';
20-
import {MatLegacyMenuModule} from '@angular/material/legacy-menu';
21-
import {MatLegacySelectModule} from '@angular/material/legacy-select';
22-
import {MatLegacyTabsModule} from '@angular/material/legacy-tabs';
20+
import {MatMenuModule} from '@angular/material/menu';
21+
import {MatSelectModule} from '@angular/material/select';
22+
import {MatTabsModule} from '@angular/material/tabs';
2323
import {MatToolbarModule} from '@angular/material/toolbar';
2424
import {CoreModule} from '../core/core_module';
2525
import {SettingsModule} from '../settings/settings_module';
@@ -47,12 +47,12 @@ import {ReloadContainer} from './reload_container';
4747
],
4848
providers: [],
4949
imports: [
50-
MatLegacyButtonModule,
50+
MatButtonModule,
5151
MatIconModule,
52-
MatLegacyTabsModule,
52+
MatTabsModule,
5353
MatToolbarModule,
54-
MatLegacySelectModule,
55-
MatLegacyMenuModule,
54+
MatSelectModule,
55+
MatMenuModule,
5656
CommonModule,
5757
CoreModule,
5858
SettingsModule,

0 commit comments

Comments
 (0)