Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Combine initializations #74

Closed
afandria opened this issue Apr 30, 2015 · 0 comments
Closed

Combine initializations #74

afandria opened this issue Apr 30, 2015 · 0 comments
Assignees
@afandria
Labels
Status: Accepted

Comments

@afandria
Copy link
Contributor

When the app initializes, we want to reload some state from the user's last run. They should be put in the same place.

  • The sidePanelWidth
  • The last namespace
  • The current view (tree/radial/grid)

In the future, this is also the correct place to put
@afandria afandria self-assigned this Apr 30, 2015
afandria added a commit that referenced this issue May 1, 2015
@afandria
nsb: Ensure that namespace is set from app load
d62550a 
Introduces services/state/service.js as the location to
manage persisted application state.

In this CL, the namespace index is loaded at the beginning of app.js
with the state service. This ensures that regardless of the initial route,
that the namespace is set to the last one the user visited.

Raises #74
Closes #48

Change-Id: I0467ea6c20bf7ce9f4e87bc4afb533655c29ae4f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@afandria afandria

Labels
Status: Accepted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@afandria