1
0
Fork 0

For #11261 - Dismisses menu when rotating to landscape (#11845)

master
Jeff Boek 2020-06-23 08:50:20 -07:00 committed by GitHub
parent 1cc259169d
commit 8ce7cf59c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 5 deletions

View File

@ -94,6 +94,7 @@ class TabTrayDialogFragment : AppCompatDialogFragment() {
tabTrayView.setTopOffset(isLandscape)
if (isLandscape) {
tabTrayView.dismissMenu()
tabTrayView.expand()
}
}

View File

@ -21,6 +21,7 @@ import kotlinx.android.synthetic.main.component_tabstray_fab.view.*
import kotlinx.coroutines.Dispatchers.Main
import kotlinx.coroutines.delay
import kotlinx.coroutines.launch
import mozilla.components.browser.menu.BrowserMenu
import mozilla.components.browser.menu.BrowserMenuBuilder
import mozilla.components.browser.menu.item.SimpleBrowserMenuItem
import mozilla.components.browser.state.selector.normalTabs
@ -54,7 +55,8 @@ class TabTrayView(
private val behavior = BottomSheetBehavior.from(view.tab_wrapper)
private var tabTrayItemMenu: TabTrayItemMenu
private val tabTrayItemMenu: TabTrayItemMenu
private var menu: BrowserMenu? = null
private var hasLoaded = false
@ -149,10 +151,9 @@ class TabTrayView(
}
view.tab_tray_overflow.setOnClickListener {
tabTrayItemMenu.menuBuilder
.build(view.context)
.show(anchor = it)
.also { pu ->
menu = tabTrayItemMenu.menuBuilder.build(container.context)
menu?.show(it)
?.also { pu ->
(pu.contentView as? CardView)?.setCardBackgroundColor(ContextCompat.getColor(
view.context,
R.color.foundation_normal_theme
@ -241,6 +242,10 @@ class TabTrayView(
behavior.setExpandedOffset(topOffset)
}
fun dismissMenu() {
menu?.dismiss()
}
companion object {
private const val DEFAULT_TAB_ID = 0
private const val PRIVATE_TAB_ID = 1