[fenix] Fixes https://github.com/mozilla-mobile/fenix/issues/2739. Update sign-in preference visibility when screen is resumed.

pull/600/head
Denys M 5 years ago committed by Emily Kager
parent 3fd777fff6
commit 2a17207de3

@ -52,7 +52,7 @@ import org.mozilla.fenix.ext.requireComponents
import org.mozilla.fenix.utils.ItsNotBrokenSnack import org.mozilla.fenix.utils.ItsNotBrokenSnack
import kotlin.coroutines.CoroutineContext import kotlin.coroutines.CoroutineContext
@SuppressWarnings("TooManyFunctions") @SuppressWarnings("TooManyFunctions", "LargeClass")
class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObserver { class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObserver {
private lateinit var job: Job private lateinit var job: Job
override val coroutineContext: CoroutineContext override val coroutineContext: CoroutineContext
@ -61,7 +61,6 @@ class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObse
override fun onCreate(savedInstanceState: Bundle?) { override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState) super.onCreate(savedInstanceState)
job = Job() job = Job()
updateSignInVisibility()
preferenceManager.sharedPreferences.registerOnSharedPreferenceChangeListener { sharedPreferences, key -> preferenceManager.sharedPreferences.registerOnSharedPreferenceChangeListener { sharedPreferences, key ->
try { try {
@ -118,6 +117,7 @@ class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObse
setupPreferences() setupPreferences()
setupAccountUI() setupAccountUI()
updateSignInVisibility()
} }
@Suppress("ComplexMethod") @Suppress("ComplexMethod")
@ -283,6 +283,7 @@ class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObse
override fun onAuthenticated(account: OAuthAccount) { override fun onAuthenticated(account: OAuthAccount) {
updateAuthState(account) updateAuthState(account)
updateSignInVisibility()
} }
override fun onError(error: Exception) { override fun onError(error: Exception) {

Loading…
Cancel
Save