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

Save the back stack of visited pages before exiting from app #448

Closed
wants to merge 1 commit into from

Conversation

RohanBh
Copy link
Contributor

@RohanBh RohanBh commented Feb 17, 2018

Fixes #398

Changes: Saved a webview state bundle to a file and restored the webview's state via that bundle on app startup. All I/O operations have been done on a new thread.

Screenshots/GIF for the change: back_forward_history

@RohanBh
Copy link
Contributor Author

RohanBh commented Feb 17, 2018

@mhutti1 I have implemented the back stack save and restore feature. The only work that remains is of showing them in proper order. I will complete the PR in a couple of days as I am busy with my exams.

@dr0pdb
Copy link
Contributor

dr0pdb commented Mar 5, 2018

@RohanBh Please update the PR incase you're free now. Ignore otherwise.

@RohanBh
Copy link
Contributor Author

RohanBh commented Mar 6, 2018

I couldn't find a direct way of doing this. Also, the method I was trying is not able to solve the issue as webview behaves differently than I thought. I might need more time.

@RohanBh RohanBh force-pushed the fix-webview-back-stack branch from 0df824b to d23b893 Compare March 10, 2018 23:20
@RohanBh RohanBh changed the title [WIP] Save the back stack of visited pages before exiting from app Save the back stack of visited pages before exiting from app Mar 10, 2018
@RohanBh
Copy link
Contributor Author

RohanBh commented Mar 10, 2018

@mhutti1 I have finally completed the PR.

@mhutti1 mhutti1 self-requested a review March 13, 2018 00:15
@RohanBh
Copy link
Contributor Author

RohanBh commented Mar 27, 2018

@mhutti1 please take a look at this PR.

*/
public static void writeBundleToStorage(final @NonNull Application app, Bundle writeBundle) {
Observable.just(writeBundle)
.observeOn(Schedulers.io())
Copy link
Contributor

Choose a reason for hiding this comment

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

4 Space continuations please. In multiple places in this PR.

@mhutti1
Copy link
Contributor

mhutti1 commented Jun 25, 2018

Closing due to inactivity.

@mhutti1 mhutti1 closed this Jun 25, 2018
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.

3 participants