For #13939: Missing empty state for downloads view
parent
0de8aedd6f
commit
86c5a38ed8
|
@ -7,8 +7,10 @@ package org.mozilla.fenix.library.downloads
|
||||||
import android.view.LayoutInflater
|
import android.view.LayoutInflater
|
||||||
import android.view.View
|
import android.view.View
|
||||||
import android.view.ViewGroup
|
import android.view.ViewGroup
|
||||||
|
import androidx.core.view.isVisible
|
||||||
import androidx.recyclerview.widget.LinearLayoutManager
|
import androidx.recyclerview.widget.LinearLayoutManager
|
||||||
import androidx.recyclerview.widget.SimpleItemAnimator
|
import androidx.recyclerview.widget.SimpleItemAnimator
|
||||||
|
import kotlinx.android.synthetic.main.component_downloads.*
|
||||||
import kotlinx.android.synthetic.main.component_downloads.view.*
|
import kotlinx.android.synthetic.main.component_downloads.view.*
|
||||||
import mozilla.components.support.base.feature.UserInteractionHandler
|
import mozilla.components.support.base.feature.UserInteractionHandler
|
||||||
import org.mozilla.fenix.R
|
import org.mozilla.fenix.R
|
||||||
|
@ -58,6 +60,8 @@ class DownloadView(
|
||||||
state.mode === DownloadFragmentState.Mode.Normal
|
state.mode === DownloadFragmentState.Mode.Normal
|
||||||
mode = state.mode
|
mode = state.mode
|
||||||
|
|
||||||
|
updateEmptyState(state.items.isNotEmpty())
|
||||||
|
|
||||||
downloadAdapter.updateMode(state.mode)
|
downloadAdapter.updateMode(state.mode)
|
||||||
downloadAdapter.updateDownloads(state.items)
|
downloadAdapter.updateDownloads(state.items)
|
||||||
|
|
||||||
|
@ -66,6 +70,14 @@ class DownloadView(
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fun updateEmptyState(userHasDownloads: Boolean) {
|
||||||
|
download_list.isVisible = userHasDownloads
|
||||||
|
download_empty_view.isVisible = !userHasDownloads
|
||||||
|
if (!userHasDownloads) {
|
||||||
|
download_empty_view.announceForAccessibility(context.getString(R.string.download_empty_message))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
override fun onBackPressed(): Boolean {
|
override fun onBackPressed(): Boolean {
|
||||||
return interactor.onBackPressed()
|
return interactor.onBackPressed()
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue