Merge pull request #2 from AnkitGhate/master

Added correct navigation action while navigating from Bookmarks to ArticleDetail
TheCodeMonks · Jul 20, 2020 · 5a0d144 · 5a0d144
2 parents f64b4ab + d2b414f
commit 5a0d144
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/app/src/main/java/www/thecodemonks/techbytes/ui/bookmarks/BookmarksFragment.kt b/app/src/main/java/www/thecodemonks/techbytes/ui/bookmarks/BookmarksFragment.kt
@@ -110,7 +110,7 @@ class BookmarksFragment : Fragment(R.layout.fragment_bookmarks) {
                 putSerializable("article", article)
             }
             findNavController().navigate(
-                R.id.action_articlesFragment_to_articleDetailsFragment,
+                R.id.action_bookmarksFragment_to_articleDetailsFragment,
                 bundle
             )
         }

diff --git a/app/src/main/res/navigation/nav_graph.xml b/app/src/main/res/navigation/nav_graph.xml
@@ -55,7 +55,11 @@
         android:id="@+id/bookmarksFragment"
         android:name="www.thecodemonks.techbytes.ui.bookmarks.BookmarksFragment"
         android:label="Bookmarks"
-        tools:layout="@layout/fragment_bookmarks" />
+        tools:layout="@layout/fragment_bookmarks" >
+        <action
+            android:id="@+id/action_bookmarksFragment_to_articleDetailsFragment"
+            app:destination="@id/articleDetailsFragment" />
+    </fragment>
     <fragment
         android:id="@+id/articleDetailsFragment"
         android:name="www.thecodemonks.techbytes.ui.details.ArticleDetailsFragment"