1914664 - [toolbar redesign] Implement CFR for address bar swipe

Status: VERIFIED FIXED

(Firefox for Android :: Toolbar, task, P2)

Description

[Roger Yang [:royang]]

This is to implement the CFR for the address bar swipe.
Figma

Comment 1

[Petru-Mugurel Lingurar [:petru]]

@Aarjav Should this ticket rather block phase 2 which would update the UX for swiping between tabs?

Comment 2

[Aarjav [:aarjav]]

No, this should block phase 1 as we have improved the way the tabs are presented to the user. Through the experiment we would be tracking if this changes tab handling for users. If the users are not aware of the swipe between tabs features, they would be informed using this CFR.

Comment 3

[Roger Yang [:royang]]

Need clarification for when this will be shown. Landscape vs portrait, new users vs existing users, old toolbar vs new toolbar.

Comment 4

[tchoh]

Attachment: Bug 1914664 - Address bar swipe CFR

Comment 5

[Roger Yang [:royang]]

Updated requirement with agreement from UX.

User has 2+ tabs open. 
AND user opens the tab tray
AND then navigates to a consecutive open tab
Only shown in normal browsing mode, not private browsing

This is so we

1. Avoid conflict with Erase action and Cookie Banner CFRs
2. Don't have to wait till page load completes before showing the CFR.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f3c65b5ac13
Address bar swipe CFR r=android-reviewers,petru

Comment 7

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/6f3c65b5ac13

Comment 8

[Delia Pop (Android QA)]

Verified as fixed in Nightly 134.0a1 from 11/20 with Samsung Galaxy S22 Ultra (Android 14), Samsung GalaxyZ Fold 4 (Android 14) and Xiaomi Redmi Note 8T (Android 11).
3 new issues related to the new CFR was found and reported: 1932362, 1932387, 1932378.