Skip to content

feat(devtools): improve transfer state tab#68535

Open
AleksanderBodurri wants to merge 1 commit into
angular:mainfrom
AleksanderBodurri:devtools-transfer-state-cleanup
Open

feat(devtools): improve transfer state tab#68535
AleksanderBodurri wants to merge 1 commit into
angular:mainfrom
AleksanderBodurri:devtools-transfer-state-cleanup

Conversation

@AleksanderBodurri
Copy link
Copy Markdown
Member

@AleksanderBodurri AleksanderBodurri commented May 3, 2026

  • promote to default-on (remove opt-in setting and gear-menu toggle)
  • replace the <pre> value cell with a collapsible JSON tree
  • add key filter and column sort (size sorts by raw bytes)
  • show an educational empty state when the app has no transfer state
  • iterate every root in the directive forest and merge their state
  • guard size calculation against undefined and circular references
  • register listener once with DestroyRef cleanup; add a loading timeout
  • distinguish array/string badge colors via theme-aware tokens
  • responsive value cell width; flex-based table scroll
  • add a component spec; seed the harness fixture with more demo data
Screenshot 2026-05-03 at 4 28 15 PM Screenshot 2026-05-03 at 4 29 15 PM

@ngbot ngbot Bot added this to the Backlog milestone May 3, 2026
@angular-robot angular-robot Bot added the detected: feature PR contains a feature commit label May 3, 2026
@AleksanderBodurri AleksanderBodurri force-pushed the devtools-transfer-state-cleanup branch from 83ab955 to 64438b1 Compare May 3, 2026 20:30
Comment thread devtools/projects/ng-devtools/src/lib/devtools-tabs/devtools-tabs.component.ts Outdated
- promote to default-on (remove opt-in setting and gear-menu toggle)
- replace the <pre> value cell with a collapsible JSON tree
- add key filter and column sort (size sorts by raw bytes)
- show an educational empty state when the app has no transfer state
- iterate every root in the directive forest and merge their state
- guard size calculation against undefined and circular references
- register listener once with DestroyRef cleanup; add a loading timeout
- distinguish array/string badge colors via theme-aware tokens
- responsive value cell width; flex-based table scroll
- add a component spec; seed the harness fixture with more demo data
@AleksanderBodurri AleksanderBodurri force-pushed the devtools-transfer-state-cleanup branch from c289bdf to b5c5bf5 Compare May 11, 2026 15:10
@AleksanderBodurri AleksanderBodurri added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels May 31, 2026
@JeanMeche JeanMeche added target: patch This PR is targeted for the next patch release and removed target: patch This PR is targeted for the next patch release labels Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: devtools detected: feature PR contains a feature commit target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants