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.

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

fix: keep track of last opened page #1452

Merged
merged 3 commits into from
Apr 27, 2020
Merged

fix: keep track of last opened page #1452

merged 3 commits into from
Apr 27, 2020

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Apr 27, 2020

Fixes #1451. Keeps track of the last opened page so every time we need to refresh because a change on the IPFS status, we can keep the user on the same page. The visibility (/blank handling) is taken care on the preload.js.

License: MIT
Signed-off-by: Henrique Dias hacdias@gmail.com

hacdias added 2 commits April 27, 2020 13:08
License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
Check if document is visible after being loaded
to avoid wasting resources.

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
@hacdias hacdias requested a review from lidel April 27, 2020 12:12
@hacdias hacdias mentioned this pull request Apr 27, 2020
22 tasks
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!
Ok to merge when CI is green.

src/webui/index.js Outdated Show resolved Hide resolved
@lidel lidel merged commit 3cb6a86 into master Apr 27, 2020
@lidel lidel deleted the fix/open-last-page branch April 27, 2020 12:54
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.

"Open Web UI at Launch" stays at /blank instead of /
2 participants