Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

"Open link in private tab" option from a custom tab does not switch to Focus #5670

Closed
delia-pop opened this issue Oct 20, 2021 · 9 comments · Fixed by #7813
Closed

"Open link in private tab" option from a custom tab does not switch to Focus #5670

delia-pop opened this issue Oct 20, 2021 · 9 comments · Fixed by #7813
Assignees
Milestone

Comments

@delia-pop
Copy link

delia-pop commented Oct 20, 2021

Steps to reproduce

  1. Set Firefox Focus as default browser.
  2. Open Gmail (or another app: Chrome, Twitter etc).
  3. Open a link from an email.
  4. From the webpage opened in a custom tab, long tap a link.
  5. Select "Open link in a private tab".
  6. From the "New private tab opened" snackbar displayed tap on the "SWITCH" button.
  7. Observe the result.

Expected behavior

As per discussion from #3753, the "Open link in a private tab" should be replaced by “Open in browser”. Once the option is selected, the tab should be immediately opened in Focus.

However, regarding the the current behavior, when tapping on the "SWITCH" button from the snackbar, the webpage should immediately be opened in Focus.

⚠️ UPDATE: expected behavior:

@jeffreygee In fact Fenix displays a different context menu for custom tabs. Should we follow the same path for Focus?
@ionutbedregeanu - Yeah, I would follow the same path for Focus as in Fenix

Actual behavior

Tapping on the "Open link in private tab" option from a custom tab does not immediately switch to Focus after selecting "SWITCH" from the snack bar.
If the user opens the Firefox Focus browser, it can be observed that the link is opened in a new tab.

Device information

  • Android device: Huawei P9 Lite (Android 8).
  • Focus version: Beta 94.0.0-beta.2 (build 352922038 GV 94.0-20211017185800)/ RC 94.0.0.
New.private.tab.opened.mp4
@mcarare mcarare added the UX label Oct 21, 2021
@mcarare
Copy link
Contributor

mcarare commented Oct 21, 2021

There are a few options on how to better handle this, but I think UX should decide.
cc @brampitoyo.

@brampitoyo
Copy link

@mcarare You are right that the “Switch” action should move the user out of the custom tab, and into the Focus app where the new tab is opened.

However, since we’re planning to deprecate multi-tasking, I would say that this issue is lower-priority to fix, compared to others.

@stale
Copy link

stale bot commented Apr 21, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Apr 21, 2022
@lobontiumira
Copy link

I was able to reproduce this issue on the latest Focus Nightly build 101.0a1 with Oppo Find X3 Lite (Android 11).
After tapping on "Switch", nothing happens. I was not redirected to Focus, I still remained in the custom tab. Opening Focus Nightly, the tab is opened, but as a user I had no idea that the action was successful.

@stale stale bot removed the wontfix label Apr 21, 2022
@lobontiumira
Copy link

This issue still reproduces on the latest Focus Nightly 104.0a1 build from 7/14 with Oppo Reno 6 (Android 12).

@lobontiumira lobontiumira changed the title "Open link in private tab" option from a custom tab does not immediately switch to Focus "Open link in private tab" option from a custom tab does not switch to Focus Jul 14, 2022
@mcarare mcarare added this to Backlog in Android Focus via automation Jul 14, 2022
@mcarare mcarare removed the waiting label Jul 14, 2022
@ionutbedregeanu ionutbedregeanu self-assigned this Jul 15, 2022
@ionutbedregeanu ionutbedregeanu moved this from Backlog to In Progress in Android Focus Jul 15, 2022
@ionutbedregeanu
Copy link
Contributor

ionutbedregeanu commented Jul 15, 2022

@jeffreygee From what i see in AC "Open link in a private tab" was implemented for normal and not for custom tabs.
This option is hidden in Fenix custom tab context menu. In fact Fenix displays a different context menu for custom tabs.
Should we follow the same path for Focus?

Fenix custom tab context menu for link and picture

Focus custom tab context menu for link and picture

@ionutbedregeanu ionutbedregeanu moved this from In Progress to Backlog in Android Focus Jul 15, 2022
@jeffreygee
Copy link

@ionutbedregeanu - Yeah, I would follow the same path for Focus as in Fenix

@mcarare mcarare self-assigned this Oct 6, 2022
@mcarare mcarare moved this from Backlog to In Progress in Android Focus Oct 6, 2022
mcarare added a commit to mcarare/focus-android that referenced this issue Oct 6, 2022
@github-actions github-actions bot added the eng:reopen-for-qa eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Oct 6, 2022
mcarare added a commit to mcarare/focus-android that referenced this issue Oct 7, 2022
mcarare added a commit to mcarare/focus-android that referenced this issue Oct 7, 2022
@mergify mergify bot closed this as completed in #7813 Oct 13, 2022
Android Focus automation moved this from In Progress to Done Oct 13, 2022
@github-actions github-actions bot reopened this Oct 13, 2022
@github-actions github-actions bot added this to the 106 milestone Oct 13, 2022
@github-actions github-actions bot added QAReady flag for test read & needed and removed eng:reopen-for-qa eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Oct 13, 2022
@mcarare mcarare moved this from Done to QA Ready in Android Focus Oct 13, 2022
@lobontiumira
Copy link

Verified as fixed on the latest Focus Nightly 107.0a1 from 10/16 with Google Pixel (Android 10), and Oppo Find X3 Lite (Android 11) - the custom tab's context menus are implemented as in Fenix, without the "Open link in private tab" option:

Untitled

Android Focus automation moved this from QA Ready to Done Oct 17, 2022
@lobontiumira lobontiumira added QAVerified and removed QAReady flag for test read & needed labels Oct 17, 2022
@lobontiumira
Copy link

Verified as fixed also on Beta 107.0b1.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Android Focus
  
Done
Development

Successfully merging a pull request may close this issue.

6 participants