Bug 1877277 - Translations add Check for isTranslationsEngineSupported

fenix/125.0
AndiAJ 3 months ago committed by mergify[bot]
parent cb7afcb65c
commit 2e5fc3b409

@ -300,7 +300,15 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler {
}
private fun initTranslationsAction(context: Context, view: View) {
if (!context.settings().enableTranslations) {
val isEngineSupported =
context.components.core.store.state.translationEngine.isEngineSupported
if (
!context.settings().enableTranslations &&
(
isEngineSupported == null ||
isEngineSupported == false
)
) {
return
}

@ -72,13 +72,16 @@ class TranslationsBinding(
// Session Translations State Behavior (Tab)
val sessionTranslationsState = state.sessionState.translationsState
if (sessionTranslationsState.isTranslated) {
val fromSelected = sessionTranslationsState.translationEngineState?.initialFromLanguage(
translateFromLanguages,
)
val toSelected = sessionTranslationsState.translationEngineState?.initialToLanguage(
translateToLanguages,
)
val fromSelected =
sessionTranslationsState.translationEngineState?.initialFromLanguage(
translateFromLanguages,
)
val toSelected =
sessionTranslationsState.translationEngineState?.initialToLanguage(
translateToLanguages,
)
if (fromSelected != null && toSelected != null) {
onStateUpdated(

@ -198,7 +198,7 @@ open class DefaultToolbarMenu(
*/
@VisibleForTesting(otherwise = PRIVATE)
fun shouldShowTranslations(): Boolean = selectedSession?.let {
context.settings().enableTranslations
context.settings().enableTranslations && store.state.translationEngine.isEngineSupported == true
} ?: false
// End of predicates //

Loading…
Cancel
Save