Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add Press Back Again Toast in Simulator#415

Open
codingblazer wants to merge 1 commit into
BuildmLearn:bug-fixes from
codingblazer:back1
Open

Add Press Back Again Toast in Simulator #415
codingblazer wants to merge 1 commit into
BuildmLearn:bug-fixes from
codingblazer:back1

Conversation

@codingblazer

@codingblazer codingblazer commented Feb 9, 2017
edited
Loading

Copy link
Copy Markdown
Contributor

Fix #368
@opticod Please Review. PR for templates sent too.
I have also handled another case - Earlier on pressing back when navigation drawer was opened caused drawer and app both to close. Now only drawer closes.

@opticod opticod left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@codingblazer , your implementation for template repo is fine and merged it, but in this code there are some problems in your implementation.

Like, go to Simulator of Info Template and then click any one of the items in list, and now click back....user is in deadlock state, neither menu back nor navigation back is working here....Please fix this.

Copy link
Copy Markdown
Contributor Author

@opticod Changes made !! Please review.

A side note - In Quiz and some templates having navigation bar, if let you open a simulator => Ques 1 appears, now if you open same from navigation bar => Press back => Nothing happens (it looks that bug is still not solved but it is !! (back press killed the duplicate instance of Ques 1)

opticod commented Sep 1, 2017

Copy link
Copy Markdown
Collaborator

Will soon start reviewing and merging the PRs. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@opticod opticod opticod requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /