Closed
Bug 2055299
Opened 1 month ago
Closed 25 days ago
[Google Lens Fenix] Opt-out screen
Categories
(Firefox for Android :: Search, task)
Tracking
()
VERIFIED
FIXED
155 Branch
People
(Reporter: harrisono, Assigned: harrisono)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][group3])
Attachments
(3 files)
Add a new dedicated onboarding for net new users and existing users where we tell existing users that they can switch back to QR code by disabling it.
Users can go directly to the settings page to turn off the Google Lens integration.
| Assignee | ||
Updated•1 month ago
|
Assignee: nobody → hoglesby
Status: NEW → ASSIGNED
Whiteboard: [fxdroid][group3]
Updated•1 month ago
|
| Assignee | ||
Comment 1•1 month ago
|
||
| Assignee | ||
Comment 2•1 month ago
|
||
Pushed by hoglesby@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/eee015fac251
https://hg.mozilla.org/integration/autoland/rev/4f55bcb465a4
Opt out screen for users on first Google Lens open r=android-reviewers,android-l10n-reviewers,parker,Roger
Status: ASSIGNED → RESOLVED
Closed: 25 days ago
status-firefox155:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 155 Branch
Comment 5•25 days ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: We need an opt-out screen for Google Lens toolbar action button as this is a change to the normal toolbar behavior and presenting this opt-out screen will help alleviate users concerned with privacy and Google.
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: no
- Steps to reproduce for manual QE testing:
- Risk associated with taking this patch: low
- Explanation of risk level: Adds an additional gated bottom sheet to the
LensCameraActivity.
Has been tested but still more risky than a simple text change. - String changes made/needed?: yes, there is a few new strings, mostly copies of other strings but with different reference ids.
- Is Android affected?: yes
Attachment #9621774 -
Flags: approval-mozilla-beta?
| Assignee | ||
Comment 6•25 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D314822
Comment 7•25 days ago
|
||
Ni camila as this requires l10n approval for new mobile strings before uplift into beta
Flags: needinfo?(cpedraza)
Comment 8•25 days ago
|
||
Thanks for the ping! diff approved! okay for l10n.
Flags: needinfo?(cpedraza)
Updated•24 days ago
|
Attachment #9621774 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•24 days ago
|
status-firefox154:
--- → fixed
Updated•24 days ago
|
Flags: qe-verify+
Comment 10•24 days ago
|
||
Verified as implemented on the Firefox for Android nightly 155.0a1 from 8/5 using a Samsung Galaxy S24 (Android 16).
Leaving the qe-verify+ label until QA can verify this on Beta 154 as well.
Comment 11•23 days ago
|
||
Verified as implemented on the Firefox for Android Beta 154.0b7 using a Pixel 6 (Android 17).
You need to log in
before you can comment on or make changes to this bug.
Description
•