For #27697 - Load icon for browser and history search immediately.
The icon was set after the screen was visible leading to a flicker. With this solution, the icon will be set before the screen is visible eliminating any visible artefacts.
This commit is contained in:
parent
4a0c1e5a8c
commit
35ab4db755
|
@ -81,6 +81,8 @@ class ToolbarView(
|
||||||
false
|
false
|
||||||
}
|
}
|
||||||
|
|
||||||
|
setDefaultIcon()
|
||||||
|
|
||||||
setOnEditListener(
|
setOnEditListener(
|
||||||
object : mozilla.components.concept.toolbar.Toolbar.OnEditListener {
|
object : mozilla.components.concept.toolbar.Toolbar.OnEditListener {
|
||||||
override fun onCancelEditing(): Boolean {
|
override fun onCancelEditing(): Boolean {
|
||||||
|
@ -110,7 +112,9 @@ class ToolbarView(
|
||||||
view.editMode()
|
view.editMode()
|
||||||
isInitialized = true
|
isInitialized = true
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private fun setDefaultIcon() {
|
||||||
val bookmarkSearchIcon =
|
val bookmarkSearchIcon =
|
||||||
AppCompatResources.getDrawable(context, R.drawable.ic_bookmarks_menu)
|
AppCompatResources.getDrawable(context, R.drawable.ic_bookmarks_menu)
|
||||||
|
|
||||||
|
|
|
@ -81,6 +81,8 @@ class ToolbarView(
|
||||||
false
|
false
|
||||||
}
|
}
|
||||||
|
|
||||||
|
setDefaultIcon()
|
||||||
|
|
||||||
setOnEditListener(
|
setOnEditListener(
|
||||||
object : mozilla.components.concept.toolbar.Toolbar.OnEditListener {
|
object : mozilla.components.concept.toolbar.Toolbar.OnEditListener {
|
||||||
override fun onCancelEditing(): Boolean {
|
override fun onCancelEditing(): Boolean {
|
||||||
|
@ -110,7 +112,9 @@ class ToolbarView(
|
||||||
view.editMode()
|
view.editMode()
|
||||||
isInitialized = true
|
isInitialized = true
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private fun setDefaultIcon() {
|
||||||
val historySearchIcon = AppCompatResources.getDrawable(context, R.drawable.ic_history)
|
val historySearchIcon = AppCompatResources.getDrawable(context, R.drawable.ic_history)
|
||||||
|
|
||||||
historySearchIcon?.let {
|
historySearchIcon?.let {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user