Add Email details screen

Change-Id: I4738e15193d34708654d1ebfbff568caf78581bf

Author: hunterstich

app/build.gradle

@@ -1,6 +1,7 @@
 apply plugin: 'com.android.application'
 apply plugin: 'kotlin-android'
 apply plugin: 'kotlin-kapt'
+apply plugin: "androidx.navigation.safeargs.kotlin"
 
 android {
     compileSdkVersion 29
@@ -35,8 +36,8 @@ dependencies {
     implementation 'androidx.legacy:legacy-support-v4:1.0.0'
     implementation 'androidx.recyclerview:recyclerview:1.1.0-alpha06'
     implementation 'androidx.core:core-ktx:1.0.2'
-    implementation 'androidx.navigation:navigation-fragment-ktx:2.1.0-alpha05'
-    implementation 'androidx.navigation:navigation-ui-ktx:2.1.0-alpha05'
+    implementation "androidx.navigation:navigation-fragment-ktx:$navigation_version"
+    implementation "androidx.navigation:navigation-ui-ktx:$navigation_version"
 
     // Glide
     implementation "com.github.bumptech.glide:glide:4.9.0"

app/src/main/java/com/materialstudies/reply/data/Email.kt

@@ -1,5 +1,6 @@
 package com.materialstudies.reply.data
 
+import androidx.annotation.DrawableRes
 import androidx.recyclerview.widget.DiffUtil
 
 /**
@@ -8,9 +9,10 @@ import androidx.recyclerview.widget.DiffUtil
 data class Email(
     val id: Int,
     val sender: String,
+    val recipient: String,
     val subject: String,
     val body: String,
-    val senderImg: Int,
+    @DrawableRes val senderResId: Int,
     val attachments: List<EmailAttachment> = emptyList(),
     var isImportant: Boolean = false,
     var isStarred: Boolean = false
@@ -20,17 +22,7 @@ data class Email(
 }
 
 object EmailDiffCallback : DiffUtil.ItemCallback<Email>() {
-    override fun areItemsTheSame(oldItem: Email, newItem: Email): Boolean = oldItem.id == newItem.id
-
-    override fun areContentsTheSame(oldItem: Email, newItem: Email): Boolean {
-        return oldItem.sender == newItem.sender &&
-            oldItem.subject == newItem.subject &&
-            oldItem.body == newItem.body &&
-            oldItem.senderImg == newItem.senderImg &&
-            oldItem.isStarred == newItem.isStarred &&
-            oldItem.isImportant == newItem.isImportant &&
-            oldItem.attachments.containsAll(newItem.attachments)
-
-    }
+    override fun areItemsTheSame(oldItem: Email, newItem: Email) = oldItem.id == newItem.id
+    override fun areContentsTheSame(oldItem: Email, newItem: Email) = oldItem == newItem
 }
 

app/src/main/java/com/materialstudies/reply/data/EmailStore.kt

@@ -13,21 +13,38 @@ class EmailStore {
         Email(
             0,
             "Google Express - 15m ago",
+            "me",
             "Package shipped!",
-            "Cucumber Mask Facial has shipped",
+            """
+                Cucumber Mask Facial has shipped.
+
+                Keep an eye out for a package to arrive between this Thursday and next Tuesday. If for any reason you don't receive your package before the end of next week, please reach out to us for details on your shipment.
+
+                As always, thank you for shopping with us and we hope you love our specially formulated Cucumber Mask!
+            """.trimIndent(),
             R.drawable.avatar_express,
             isStarred = true
         ),
         Email(
             1,
             "Ali Connors - 25m ago",
+            "me",
             "Brunch this weekend?",
-            "I'll be in your neighborhood doing errands...",
+            """
+                I'll be in your neighborhood doing errands and was hoping to catch you for a coffee this Saturday. If you don't have anything scheduled, it would be great to see you! It feels like its been forever.
+
+                If we do get a chance to get together, remind me to tell you about Kim. She stopped over at the house to say hey to the kids and told me all about her trip to Mexico.
+
+                Talk to you soon,
+
+                Ali
+            """.trimIndent(),
             R.drawable.avatar_1
         ),
         Email(
             2,
             "Sandra Adams - 6 hrs ago",
+            "me",
             "Bonjour from Paris",
             "Here are some great shots from my trip...",
             R.drawable.avatar_2,
@@ -42,28 +59,46 @@ class EmailStore {
         Email(
             3,
             "Trevor Hansen - 12 hrs ago",
+            "me",
             "High school reunion?",
-            "",
+            """
+                Hi friends,
+
+                I was at the grocery store on Sunday night.. when I ran into Genie Williams! I almost didn't recognize her afer 20 years!
+
+                Anyway, it turns out she is on the organizing committee for the high school reunion this fall. I don't know if you were planning on going or not, but she could definitely use our help in trying to track down lots of missing alums. If you can make it, we're doing a little phone-tree party at her place next Saturday, hoping that if we can find one person, thee more will...
+            """.trimIndent(),
             R.drawable.avatar_3
         ),
         Email(
             4,
             "Britta Holt - 18 hrs ago",
+            "me",
             "Brazil trip",
-            "Thought we might be able to go over some details about our upcoming vacation...",
+            """
+                Thought we might be able to go over some details about our upcoming vacation.
+
+                I've been doing a bit of research and have come across a few paces in Northern Brazil that I think we should check out. One, the north has some of the most predictable wind on the planet. I'd love to get out on the ocean and kitesurf for a couple of days if we're going to be anywhere near or around Taiba. I hear it's beautiful there and if you're up for it, I'd love to go. Other than that, I haven't spent too much time looking into places along our road trip route. I'm assuming we can find places to stay and things to do as we drive and find places we think look interesting. But... I know you're more of a planner, so if you have ideas or places in mind, lets jot some ideas down!
+
+                Maybe we can jump on the phone later today if you have a second.
+
+                I've been doing a bit of research and have come across a few paces in Northern Brazil that I think we should check out. One, the north has some of the most predictable wind on the planet. I'd love to get out on the ocean and kitesurf for a couple of days if we're going to be anywhere near or around Taiba. I hear it's beautiful there and if you're up for it, I'd love to go. Other than that, I haven't spent too much time looking into places along our road trip route. I'm assuming we can find places to stay and things to do as we drive and find places we think look interesting. But... I know you're more of a planner, so if you have ideas or places in mind, lets jot some ideas down!
+            """.trimIndent(),
             R.drawable.avatar_4,
             isStarred = true
         ),
         Email(
             5,
             "Frank Hawkins - 20 hrs ago",
+            "me",
             "Update to Your Itinerary",
             "",
             R.drawable.avatar_5
         ),
         Email(
             6,
             "Britta Holt - 21 hrs ago",
+            "me",
             "Recipe to try",
             "Raspberry Pie: We should make this pie recipe tonight! The filling is " +
                 "very quick to put together.",
@@ -72,6 +107,7 @@ class EmailStore {
         Email(
             7,
             "Google Express - 22 hrs ago",
+            "me",
             "Delivered",
             "Your shoes should be waiting for you at home!",
             R.drawable.avatar_7
@@ -86,6 +122,10 @@ class EmailStore {
         _emails.value = allEmails
     }
 
+    fun get(id: Int): Email? {
+        return allEmails.firstOrNull { it.id == id }
+    }
+
     fun delete(id: Int) {
         allEmails.removeAll { it.id == id }
         _emails.value = allEmails

app/src/main/java/com/materialstudies/reply/ui/MainActivity.kt

@@ -3,8 +3,14 @@ package com.materialstudies.reply.ui
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
 import android.view.MenuItem
+import android.view.View
+import androidx.annotation.MenuRes
 import androidx.appcompat.app.AppCompatDelegate
 import androidx.appcompat.widget.Toolbar
+import androidx.core.content.ContextCompat
+import androidx.navigation.NavController
+import androidx.navigation.NavDestination
+import androidx.navigation.findNavController
 import com.materialstudies.reply.R
 import com.materialstudies.reply.databinding.ActivityMainBinding
 import com.materialstudies.reply.ui.nav.AlphaSlideAction
@@ -13,7 +19,9 @@ import com.materialstudies.reply.ui.nav.QuarterRotateSlideAction
 import com.materialstudies.reply.ui.nav.ShowHideFabStateAction
 import com.materialstudies.reply.util.contentView
 
-class MainActivity : AppCompatActivity(), Toolbar.OnMenuItemClickListener {
+class MainActivity : AppCompatActivity(),
+    Toolbar.OnMenuItemClickListener,
+    NavController.OnDestinationChangedListener {
 
     private val binding: ActivityMainBinding by contentView(R.layout.activity_main)
 
@@ -23,31 +31,102 @@ class MainActivity : AppCompatActivity(), Toolbar.OnMenuItemClickListener {
     }
 
     private fun setUpBottomNavigationAndFab() {
-        binding.fab.setShowMotionSpecResource(R.animator.fab_show)
-        binding.fab.setHideMotionSpecResource(R.animator.fab_hide)
+        // Wrap binding.run to ensure ContentViewBindingDelegate is calling this Activity's
+        // setContentView before accessing views
+        binding.run {
+            findNavController(R.id.nav_host_fragment).addOnDestinationChangedListener(
+                this@MainActivity
+            )
+        }
+
+        // Set a custom animation for showing and hiding the FAB
+        binding.fab.apply {
+            setShowMotionSpecResource(R.animator.fab_show)
+            setHideMotionSpecResource(R.animator.fab_hide)
+        }
 
         binding.bottomNavigationDrawer.apply {
             addOnSlideAction(QuarterRotateSlideAction(binding.bottomAppBarChevron))
             addOnSlideAction(AlphaSlideAction(binding.bottomAppBarTitle, true))
             addOnStateChangedAction(ShowHideFabStateAction(binding.fab))
             addOnStateChangedAction(ChangeSettingsMenuStateAction { showSettings ->
+                // Toggle between the current destination's BAB menu and the menu which should
+                // be displayed when the BottomNavigationDrawer is open.
                 binding.bottomAppBar.replaceMenu(if (showSettings) {
                     R.menu.bottom_app_bar_settings_menu
                 } else {
-                    R.menu.bottom_app_bar_home_menu
+                    getBottomAppBarMenuForDestination()
                 })
             })
         }
 
-        binding.bottomAppBar.setNavigationOnClickListener {
-            binding.bottomNavigationDrawer.toggle()
+        // Set up the BottomAppBar menu
+        binding.bottomAppBar.apply {
+            overflowIcon = ContextCompat.getDrawable(
+                this@MainActivity,
+                R.drawable.ic_more_vert_on_branded
+            )
+            setNavigationOnClickListener {
+                binding.bottomNavigationDrawer.toggle()
+            }
+            setOnMenuItemClickListener(this@MainActivity)
         }
-        binding.bottomAppBar.setOnMenuItemClickListener(this)
+
+        // Set up the BottomNavigationDrawer's open/close affordance
         binding.bottomAppBarContentContainer.setOnClickListener {
             binding.bottomNavigationDrawer.toggle()
         }
     }
 
+    override fun onDestinationChanged(
+        controller: NavController,
+        destination: NavDestination,
+        arguments: Bundle?
+    ) {
+        // Set the configuration of the BottomAppBar and FAB based on the current destination.
+        when (destination.id) {
+            R.id.homeFragment ->
+                setBottomAppBarForHome(getBottomAppBarMenuForDestination(destination))
+            R.id.emailFragment ->
+                setBottomAppBarForEmail(getBottomAppBarMenuForDestination(destination))
+        }
+    }
+
+    /**
+     * Helper function which returns the menu which should be displayed for the current
+     * destination.
+     *
+     * Used both when the destination has changed, centralizing destination-to-menu mapping, as
+     * well as switching between the alternate menu used when the BottomNavigationDrawer is
+     * open and closed.
+     */
+    @MenuRes
+    private fun getBottomAppBarMenuForDestination(destination: NavDestination? = null): Int {
+        val dest = destination ?: findNavController(R.id.nav_host_fragment).currentDestination
+        return when (dest?.id) {
+            R.id.homeFragment -> R.menu.bottom_app_bar_home_menu
+            R.id.emailFragment -> R.menu.bottom_app_bar_email_menu
+            else -> R.menu.bottom_app_bar_home_menu
+        }
+    }
+
+    private fun setBottomAppBarForHome(@MenuRes menuRes: Int) {
+        binding.run {
+            fab.setImageResource(R.drawable.ic_edit_on_secondary)
+            bottomAppBar.replaceMenu(menuRes)
+            bottomAppBarTitle.visibility = View.VISIBLE
+        }
+    }
+
+    private fun setBottomAppBarForEmail(@MenuRes menuRes: Int) {
+        binding.run {
+            fab.setImageResource(R.drawable.ic_reply_all_on_secondary)
+            bottomAppBar.replaceMenu(menuRes)
+            bottomAppBarTitle.visibility = View.INVISIBLE
+        }
+    }
+
+
     override fun onMenuItemClick(item: MenuItem?): Boolean {
         when (item?.itemId) {
             R.id.menu_theme -> {
@@ -64,6 +143,9 @@ class MainActivity : AppCompatActivity(), Toolbar.OnMenuItemClickListener {
         }.show(supportFragmentManager, null)
     }
 
+    /**
+     * Set this Activity's night mode based on a user's in-app selection.
+     */
     private fun onDarkThemeMenuItemSelected(itemId: Int): Boolean {
         val nightMode = when (itemId) {
             R.id.menu_light -> AppCompatDelegate.MODE_NIGHT_NO

app/src/main/java/com/materialstudies/reply/ui/common/EmailAttachmentAdapter.kt

@@ -0,0 +1,45 @@
+package com.materialstudies.reply.ui.common
+
+import android.view.LayoutInflater
+import android.view.ViewGroup
+import androidx.databinding.DataBindingUtil
+import androidx.recyclerview.widget.RecyclerView
+import com.materialstudies.reply.data.EmailAttachment
+
+/**
+ * Generic RecyclerView.Adapter to display [EmailAttachment]s.
+ */
+abstract class EmailAttachmentAdapter : RecyclerView.Adapter<EmailAttachmentViewHolder>() {
+
+    private var list: List<EmailAttachment> = emptyList()
+
+    override fun getItemCount() = list.size
+
+    override fun getItemViewType(position: Int) = getLayoutIdForPosition(position)
+
+    fun submitList(attachments: List<EmailAttachment>) {
+        list = attachments
+        notifyDataSetChanged()
+    }
+
+    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): EmailAttachmentViewHolder {
+        return EmailAttachmentViewHolder(DataBindingUtil.inflate(
+            LayoutInflater.from(parent.context),
+            viewType,
+            parent,
+            false
+        ))
+    }
+
+    override fun onBindViewHolder(holder: EmailAttachmentViewHolder, position: Int) {
+        holder.bind(list[position])
+    }
+
+    /**
+     * Clients should implement this method to determine what layout is inflated for a given
+     * position. The layout must include a data parameter named 'emailAttachment' with a type
+     * of [EmailAttachment].
+     */
+    abstract fun getLayoutIdForPosition(position: Int): Int
+
+}

app/src/main/java/com/materialstudies/reply/ui/common/EmailAttachmentViewHolder.kt

@@ -0,0 +1,22 @@
+package com.materialstudies.reply.ui.common
+
+import androidx.databinding.ViewDataBinding
+import androidx.databinding.library.baseAdapters.BR
+import androidx.recyclerview.widget.RecyclerView
+import com.materialstudies.reply.data.EmailAttachment
+
+/**
+ * Generic RecyclerView.ViewHolder which is able to bind layouts which expose a variable
+ * for an [EmailAttachment].
+ */
+class EmailAttachmentViewHolder(
+    private val binding: ViewDataBinding
+) : RecyclerView.ViewHolder(binding.root) {
+
+    fun bind(attachment: EmailAttachment) {
+        binding.run {
+            binding.setVariable(BR.emailAttachment, attachment)
+            executePendingBindings()
+        }
+    }
+}