removed the toast message and put the args bundle to be instantiated in the share intent

kevinmainairungu · kevinmainairungu · commit 8df1571d92e0 · 2020-05-03T17:33:06.000+03:00
diff --git a/app/src/main/java/com/example/android/navigation/GameWonFragment.kt b/app/src/main/java/com/example/android/navigation/GameWonFragment.kt
@@ -38,9 +38,6 @@ class GameWonFragment : Fragment() {
                     GameWonFragmentDirections.actionGameWonFragmentToGameFragment())
         }
 
-        Toast.makeText(context,
-                "NumCorrect: ${args.numCorrect}, NumQuestions: ${args.numQuestions}",
-                Toast.LENGTH_LONG).show()
         setHasOptionsMenu(true)
         return binding.root
     }
@@ -62,4 +59,11 @@ class GameWonFragment : Fragment() {
     private fun shareSuccess() {
         startActivity(getShareIntent())
     }
+
+    override fun onOptionsItemSelected(item: MenuItem?): Boolean {
+        when (item!!.itemId) {
+            R.id.share -> shareSuccess()
+        }
+        return super.onOptionsItemSelected(item)
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -38,9 +38,6 @@ class GameWonFragment : Fragment() {`
`38`	`38`	`GameWonFragmentDirections.actionGameWonFragmentToGameFragment())`
`39`	`39`	`}`
`40`	`40`
`41`		`- Toast.makeText(context,`
`42`		`- "NumCorrect: ${args.numCorrect}, NumQuestions: ${args.numQuestions}",`
`43`		`- Toast.LENGTH_LONG).show()`
`44`	`41`	`setHasOptionsMenu(true)`
`45`	`42`	`return binding.root`
`46`	`43`	`}`
`@@ -62,4 +59,11 @@ class GameWonFragment : Fragment() {`
`62`	`59`	`private fun shareSuccess() {`
`63`	`60`	`startActivity(getShareIntent())`
`64`	`61`	`}`
	`62`	`+`
	`63`	`+ override fun onOptionsItemSelected(item: MenuItem?): Boolean {`
	`64`	`+ when (item!!.itemId) {`
	`65`	`+ R.id.share -> shareSuccess()`
	`66`	`+ }`
	`67`	`+ return super.onOptionsItemSelected(item)`
	`68`	`+ }`
`65`	`69`	`}`