Skip to content

Commit da3c216

Browse files
committed
fix: naming and removed trivial method
1 parent d2b0df0 commit da3c216

File tree

3 files changed

+5
-13
lines changed

3 files changed

+5
-13
lines changed

projects/distributed-tracing/src/shared/dashboard/widgets/table/table-widget-renderer.component.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import {
1717
TableColumnConfig,
1818
TableDataSource,
1919
TableFilter,
20-
TableMode,
2120
TableRow,
2221
TableSelectionMode,
2322
TableStyle,
@@ -72,7 +71,7 @@ import { TableWidgetModel } from './table-widget.model';
7271
[ngClass]="{ 'header-margin': this.model.header?.topMargin }"
7372
[columnConfigs]="this.columnConfigs$ | async"
7473
[metadata]="this.metadata$ | async"
75-
[mode]="this.activeMode"
74+
[mode]="this.model.mode"
7675
[selectionMode]="this.model.getSelectionMode()"
7776
[display]="this.model.style"
7877
[data]="this.data$ | async"
@@ -97,7 +96,6 @@ export class TableWidgetRendererComponent
9796
implements OnInit {
9897
public filterItems: ToggleItem<TableWidgetFilterModel>[] = [];
9998
public viewItems: ToggleItem<string>[] = [];
100-
public activeMode!: string;
10199

102100
public selectFilterItems$!: Observable<SelectFilter[]>;
103101

@@ -124,8 +122,6 @@ export class TableWidgetRendererComponent
124122
public ngOnInit(): void {
125123
super.ngOnInit();
126124

127-
this.onModeChange(this.model.mode);
128-
129125
this.metadata$ = this.getScopeAttributes();
130126
this.columnConfigs$ = (isNonEmptyString(this.model.id)
131127
? this.preferenceService.get<TableColumnConfig[]>(this.model.id, [])
@@ -328,10 +324,6 @@ export class TableWidgetRendererComponent
328324
this.searchFilterSubject.next([searchFilter]);
329325
}
330326

331-
public onModeChange(mode: TableMode): void {
332-
this.activeMode = mode;
333-
}
334-
335327
public onViewChange(view: string): void {
336328
this.model.setView(view);
337329
this.columnConfigs$ = this.getColumnConfigs();

projects/distributed-tracing/src/shared/dashboard/widgets/table/table-widget-view-toggle.model.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,11 @@ export class TableWidgetViewToggleModel extends TableWidgetBaseModel {
4141
}
4242

4343
public getViewOptions(): string[] {
44-
return this.views.map(v => v.view);
44+
return this.views.map(v => v.label);
4545
}
4646

4747
private createDelegate(view: string): TableWidgetModel | undefined {
48-
const found = this.views.find(v => v.view.toLowerCase() === view.toLowerCase());
48+
const found = this.views.find(v => v.label.toLowerCase() === view.toLowerCase());
4949

5050
return found ? this.api.createChild<TableWidgetModel>(found.template) : undefined;
5151
}

projects/distributed-tracing/src/shared/dashboard/widgets/table/table-widget-view.model.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@ import { Model, ModelJson, ModelProperty, STRING_PROPERTY } from '@hypertrace/hy
66
})
77
export class TableWidgetViewModel {
88
@ModelProperty({
9-
key: 'view',
9+
key: 'label',
1010
type: STRING_PROPERTY.type
1111
})
12-
public view!: string;
12+
public label!: string;
1313

1414
@ModelProperty({
1515
key: 'template',

0 commit comments

Comments
 (0)