brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.86k stars 2.33k forks source link

disable auto translate in Brave Translate #27040

Closed rebron closed 1 year ago

rebron commented 1 year ago

Description

Temporarily disable auto translate in Brave Translate by hiding the ui checkbox and menu option as well as disabling the setting for those who have previously checked the setting to on.

Place auto translate feature behind a flag that where rollout of the feature can be controlled via griffin.

Expected result:

  1. Visit a page that invokes a Brave Translate
  2. Ensure the checkbox and options to translate are not available.

Current result:

Screenshot 2022-11-23 at 3 22 41 PM screenshot_2022-11-23_at_1 39 33_pm

Desktop Brave version:

Targeted for 1.46.x

Android Device details:

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

kjozwiak commented 1 year ago

The above requires 1.46.138 or higher for 1.46.x verification 👍 I think one platform for both Desktop & Android is enough. Maybe run through macOS as I went through Win 11 x64 on Nightly as per https://github.com/brave/brave-variations/pull/479#issuecomment-1340356174. As per @rebron, we won't be pushing the Griffin study on the Release channel until we have more information from the Nightly & Beta channels.

@brave/qa-team can follow the STR/Cases outlined via https://github.com/brave/brave-variations/pull/479#issuecomment-1340356174 & https://github.com/brave/brave-variations/pull/479#issuecomment-1340400758 but use the flag mentioned via https://github.com/brave/brave-core/pull/16185#issue-1471452908 rather than using Griffin staging like I did.

LaurenWags commented 1 year ago

Verified with

Brave | 1.46.138 Chromium: 108.0.5359.94 (Official Build) (x86_64)
-- | --
Revision | 713576b895246504ccc6b92c2fb8ce2d60194074-refs/branch-heads/5359_71@{#3}
OS | macOS Version 12.6.1 (Build 21G217)

Per https://github.com/brave/brave-browser/issues/27040#issuecomment-1340405222, verifying cases from https://github.com/brave/brave-variations/pull/479#issuecomment-1340356174 using flag mentioned in https://github.com/brave/brave-core/pull/16185#issue-1471452908.

Note - tested with staging env for Brave Translate due to production server side problems. See https://bravesoftware.slack.com/archives/C03MMJ4CHEZ/p1670421276597499 for additional context.

Test Case #1 - AutoTranslateDisabled once Always translate selected via context menu

Example Example Example Example Example
1 2 3 4 5

Test Case #2 - AutoTranslateDisabled once Always translate selected once it appears automatically in the button

Example Example Example Example Example
1 2 3 4 5

Test Case #3 - Clean Install/Profile

Example Example Example
1 2 3
kjozwiak commented 1 year ago

Probably should exclude this from the notes as this will be enabled a bit after the release. @LaurenWags @rebron please re-add if you feel differently.

srirambv commented 1 year ago

Verification passed on Oppo Reno 5 with Android 12 running 1.46.138 x64 build

Test Case 1: Auto Translate Disabled via flag

Test Case 2: Page Auto-Translate checked and then disabled via flag

Test Case 1 Test Case 2