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

Provide example of showing and hiding Fragment with NavigationView #1113

Merged
merged 1 commit into from
Aug 3, 2018

Conversation

danesfeder
Copy link
Contributor

Closes #1109

This PR iterates on the Fragment example we provide in the test app. Now we demonstrate how to show / hide a Fragment inflating a NavigationView with a FragmentTransaction.

ezgif com-video-to-gif

Copy link
Contributor

@Guardiola31337 Guardiola31337 left a comment

Choose a reason for hiding this comment

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

Other than 👇 minor comment, this looks good to me 🚢

@@ -258,6 +260,7 @@ private void addMilestones(NavigationViewOptions options) {
private ProgressChangeListener progressChangeListener = new ProgressChangeListener() {
@Override
public void onProgressChange(Location location, RouteProgress routeProgress) {
Timber.d("NAV_DEBUG - onProgressChange");
Copy link
Contributor

Choose a reason for hiding this comment

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

I imagine this slipped in by mistake (debugging purposes). We should remove this log.

@danesfeder danesfeder force-pushed the dan-second-init branch 3 times, most recently from 8d32915 to 1c74b36 Compare August 3, 2018 18:32
@danesfeder danesfeder merged commit 4954c7a into master Aug 3, 2018
@danesfeder danesfeder deleted the dan-second-init branch August 3, 2018 20:04
@Guardiola31337 Guardiola31337 mentioned this pull request Aug 10, 2018
11 tasks
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.

2 participants