-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Analysis view: use initialOptions
to save & restore view state on navigation
#1744
Conversation
initialOptions
to save & restore view state on navigation
f65e813
to
0fd7c99
Compare
@JAllen42 or @ChrisPaulBennett could 1 of you review this? |
@MetRonnie I'm on it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Use a template ref to ensure we only teleport once analysis view has been mounted
a9fc686
to
b086ef3
Compare
6aeea88
to
528ec15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@MetRonnie, @markgrahamdawson, two approvals, good to merge? |
These changes build on the work undertaken to save & restore workspace layout on navigation #1664
Which is part of a larger issue #662
There was an equivalent pull request for the log view #1688
There is an equivalent pull request for the tree view which this builds on: #1705
There is an equivalent pull request for the table view which this builds on: #1711
There is an equivalent pull request for the graph view which this builds on: #1717
Check List
CONTRIBUTING.md
and added my name as a Code Contributor.setup.cfg
(andconda-environment.yml
if present).CHANGES.md
entry included if this is a change that can affect users?.?.x
branch.