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

[Meta] Custom Tab Support #222

Closed
bbinto opened this issue Jan 21, 2019 · 13 comments
Closed

[Meta] Custom Tab Support #222

bbinto opened this issue Jan 21, 2019 · 13 comments
Labels
Feature:CustomTabs needs:ac Needs Android Component Work
Milestone

Comments

@bbinto
Copy link
Contributor

bbinto commented Jan 21, 2019

Why/User Benefit/User Problem

As a user I want to be able to tap on a link within an app, launch a site with my default browser (Fenix) without leaving the app, and close the site to return to the app.

What / Requirements

UI mockups
~~Android Component: https://github.com/mozilla~~mobile/android-components/issues/1398

Acceptance Criteria (how do I know when I’m done?)

-Customizable tab allowing to add toolbarcolour and menu items.
Allow for prestarting Fenix and pre-fetching content for faster loading.

┆Issue is synchronized with this Jira Task

@bbinto bbinto added this to the MVP Metas milestone Jan 21, 2019
@jonalmeida
Copy link
Contributor

The original link was pointing to a sub-task for an r-b user story (which is similar-ish to a meta): mozilla-mobile/reference-browser#137

@bbinto bbinto added the Epic label Jan 23, 2019
@bbinto
Copy link
Contributor Author

bbinto commented Jan 23, 2019

Need to chat with UX (@lime124) if we need any specific UX for this or can we use what A/C has done, @st3fan should have a bit more information

@st3fan
Copy link
Contributor

st3fan commented Jan 23, 2019

It is best to look at the Reference Browser to see the current UI to find out if that is adequate for a first release. There are probably some limitations about what we can do UI-wise I think, so we definitely need to get together to look at current and wanted designs.

@bbinto
Copy link
Contributor Author

bbinto commented Jan 24, 2019

Feature parity to Focus (minus the issues)

@bbinto
Copy link
Contributor Author

bbinto commented Jan 24, 2019

UX to confirm how this will work for sessions
(cc @lime124 )

@shorlander
Copy link

It is best to look at the Reference Browser to see the current UI to find out if that is adequate for a first release. There are probably some limitations about what we can do UI-wise I think, so we definitely need to get together to look at current and wanted designs.

When I try and use the Reference Browser for this it just loads the link in the browser. Not a Custom Tab.

@st3fan am I doing something wrong?

@jonalmeida
Copy link
Contributor

@shorlander Sorry about that, Custom Tabs hasn't landed in the Reference Browser because of an integration bug that I'm currently looking into (see: mozilla-mobile/reference-browser#487).

@jonalmeida
Copy link
Contributor

@shorlander Custom Tabs has landed in R-B so feel free to have a look.

@bbinto bbinto added the must label Jan 30, 2019
@st3fan
Copy link
Contributor

st3fan commented Feb 1, 2019

@shorlander a good app to test this with is the official Reddit app. Custom Tabs should open with a blue location bar. Another app where this works well is GMail, which should open up a tab with a red location bar.

I don't think Twitter properly supports this - it seems links from Twitter are opened directly in the R-B by switching apps not by opening a custom tab.

@sblatz
Copy link
Contributor

sblatz commented Mar 13, 2019

As a part of this, let's make sure we test our menu options work properly in a custom tab. Find in page and request desktop site do not function correctly as is.

Breakout ticket: #1007

@bbinto
Copy link
Contributor Author

bbinto commented Mar 14, 2019

I remember @vesta0 mentioning we had some custom tabs issue in Focus, let's make sure we are addressing them / being aware of them for Fenix as well.

sblatz added a commit to sblatz/fenix that referenced this issue Mar 14, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Mar 14, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Mar 14, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Mar 14, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Mar 15, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Mar 15, 2019
@vesta0 vesta0 removed the P2 Upcoming release label Mar 18, 2019
@jonalmeida
Copy link
Contributor

As a part of this, let's make sure we test our menu options work properly in a custom tab. Find in page and request desktop site do not function correctly as is.

I've filed an issue for this: mozilla-mobile/android-components#2387

@vesta0 vesta0 added the needs:ac Needs Android Component Work label Apr 1, 2019
@vesta0 vesta0 removed their assignment Apr 24, 2019
@vesta0
Copy link
Collaborator

vesta0 commented Jul 3, 2019

All issues listed under this meta have been implemented.

@vesta0 vesta0 closed this as completed Jul 3, 2019
@data-sync-user data-sync-user changed the title [Meta] Custom Tab Support FNX2-17773 ⁃ [Meta] Custom Tab Support Aug 5, 2020
@data-sync-user data-sync-user changed the title FNX2-17773 ⁃ [Meta] Custom Tab Support FNX3-16152 ⁃ [Meta] Custom Tab Support Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-16152 ⁃ [Meta] Custom Tab Support FNX-5523 ⁃ [Meta] Custom Tab Support Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-5523 ⁃ [Meta] Custom Tab Support [Meta] Custom Tab Support May 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature:CustomTabs needs:ac Needs Android Component Work
Projects
None yet
Development

No branches or pull requests

8 participants