Skip to content

Commit a3f808d

Browse files
Avcharovatscott
authored andcommitted
fix: remove refresh button from transfer state tab (#63592)
TransferState is only written into the DOM once during SSR and is not kept in sync with the runtime state on the client. Pressing the refresh button always re-reads the initial serialized script tag, which never changes after bootstrap. PR Close #63592
1 parent 951701d commit a3f808d

File tree

3 files changed

+0
-21
lines changed

3 files changed

+0
-21
lines changed

devtools/projects/ng-devtools/src/lib/devtools-tabs/transfer-state/transfer-state.component.html

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,6 @@ <h2>
44
<mat-icon>swap_horiz</mat-icon>
55
Transfer State
66
</h2>
7-
<div class="actions">
8-
<button ng-button btnType="icon" (click)="refresh()" matTooltip="Refresh transfer state">
9-
<mat-icon>refresh</mat-icon>
10-
</button>
11-
</div>
127
</div>
138

149
@if (isLoading()) {

devtools/projects/ng-devtools/src/lib/devtools-tabs/transfer-state/transfer-state.component.scss

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -26,18 +26,6 @@
2626
height: 1.25rem;
2727
}
2828
}
29-
30-
.actions {
31-
display: flex;
32-
gap: 0.5rem;
33-
align-items: center;
34-
35-
button {
36-
display: flex;
37-
align-items: center;
38-
gap: 0.25rem;
39-
}
40-
}
4129
}
4230

4331
.loading {

devtools/projects/ng-devtools/src/lib/devtools-tabs/transfer-state/transfer-state.component.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -142,10 +142,6 @@ export class TransferStateComponent {
142142
}
143143
}
144144

145-
refresh(): void {
146-
this.loadTransferState();
147-
}
148-
149145
isValueLong(element: HTMLElement, isExpanded: boolean = false): boolean {
150146
if (isExpanded) return true;
151147

0 commit comments

Comments
 (0)