blallo
/
fenix
1
0
Fork 0
Code Releases Activity

For #10453: Do not set item decoration when creating ViewHolder

master
Jonathan Almeida 2020-05-07 11:13:43 -04:00 committed by Jonathan Almeida
parent 77705d1a27
commit 57eb2b9549
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
app/src/main/java/org/mozilla/fenix/HomeActivity.kt
View File

@ -223,15 +223,6 @@ open class HomeActivity : LocaleAwareAppCompatActivity() {
TabsTray::class.java.name -> { TabsTray::class.java.name -> {
val layout = LinearLayoutManager(context) val layout = LinearLayoutManager(context)
val adapter = TabsAdapter { parentView, tabsTray -> val adapter = TabsAdapter { parentView, tabsTray ->
val decoration = DividerItemDecoration(
context,
DividerItemDecoration.VERTICAL
)
val drawable = AppCompatResources.getDrawable(context, R.drawable.tab_tray_divider)
drawable?.let {
decoration.setDrawable(it)
tabsTray.addItemDecoration(decoration)
}
DefaultTabViewHolder( DefaultTabViewHolder(
LayoutInflater.from(parentView.context).inflate( LayoutInflater.from(parentView.context).inflate(
@ -241,7 +232,17 @@ open class HomeActivity : LocaleAwareAppCompatActivity() {
tabsTray tabsTray
) )
} }
BrowserTabsTray(context, attrs, tabsAdapter = adapter, layout = layout) val tray = BrowserTabsTray(context, attrs, tabsAdapter = adapter, layout = layout)
val decoration = DividerItemDecoration(
context,
DividerItemDecoration.VERTICAL
)
val drawable = AppCompatResources.getDrawable(context, R.drawable.tab_tray_divider)
drawable?.let {
decoration.setDrawable(it)
tray.addItemDecoration(decoration)
}
tray
} }
else -> super.onCreateView(parent, name, context, attrs) else -> super.onCreateView(parent, name, context, attrs)
} }