Skip to content

ui: Persist expanded iciclegraph state in URL#5579

Merged
manojVivek merged 16 commits into
mainfrom
persist-expanded-flamechart-to-url
Apr 8, 2025
Merged

ui: Persist expanded iciclegraph state in URL#5579
manojVivek merged 16 commits into
mainfrom
persist-expanded-flamechart-to-url

Conversation

@manojVivek

Copy link
Copy Markdown
Contributor

This change persists the stack expansion state to the URL when a user interacts with the Iciclegraph, so that it can be easily shared.

@manojVivek
manojVivek requested a review from a team as a code owner April 3, 2025 06:33
@alwaysmeticulous

alwaysmeticulous Bot commented Apr 7, 2025

Copy link
Copy Markdown

🤖 Meticulous evaluated 50 user flows and took 323 visual snapshots. Meticulous has not yet run on c5734db of the main branch and so there was nothing to compare against.

If you recently setup Meticulous, this is expected. Meticulous will start reporting comparisons for new pull requests after the next commit to the main branch.

Last updated for commit 576b720. This comment will update as new commits are pushed.

@brancz brancz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

@manojVivek
manojVivek merged commit e18dc61 into main Apr 8, 2025
@manojVivek
manojVivek deleted the persist-expanded-flamechart-to-url branch April 8, 2025 11:45
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