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

[Bug] Open in app banner #16133

Closed
topotropic opened this issue Oct 22, 2020 · 5 comments
Closed

[Bug] Open in app banner #16133

topotropic opened this issue Oct 22, 2020 · 5 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. Feature:OpenInApp intents S3 Blocks non-critical functionality and a work around exists

Comments

@topotropic
Copy link

topotropic commented Oct 22, 2020

Steps to reproduce

  • Use a fresh install or clear the cache
  • Open Firefox and search for something with Google
  • "Open in app" banner appears on google search result page
  • When I turn "Open in app" on and try the search again, nothing happens, no app opens

Expected behavior

I don't see a banner

Actual behavior

I see a banner on top of my google search results

Device information

  • Android device: ?
  • Fenix version: ?

┆Issue is synchronized with this Jira Task

@topotropic topotropic added 🐞 bug Crashes, Something isn't working, .. Feature:OpenInApp intents labels Oct 22, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Oct 22, 2020
@kbrosnan kbrosnan removed the needs:triage Issue needs triage label Oct 22, 2020
@liuche
Copy link
Contributor

liuche commented Oct 22, 2020

This should be solvable in AC, we already have some ability to exclude the banner for some sites.

@liuche
Copy link
Contributor

liuche commented Oct 22, 2020

cc @rocketsroger we should do this bc it affects search.

@rocketsroger
Copy link
Contributor

@topotropic is this still reproducible? I can see banner is other sites such as twitter but not in google search.

@rocketsroger
Copy link
Contributor

While debugging I noticed that a simpler if condition can make maintenance easier. Also added some tests. Still cannot reproduce the issue.

@rocketsroger rocketsroger added the S3 Blocks non-critical functionality and a work around exists label Dec 7, 2020
@rocketsroger
Copy link
Contributor

I'll close this issue since I can't reproduce it. I've cleaned up the if condition so it's now easier to see that unless there is an app (non browser) that registers to open google.com, the app banner should now show in the search page. Please reopen if reproducible.

pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. Feature:OpenInApp intents S3 Blocks non-critical functionality and a work around exists
Projects
None yet
Development

No branches or pull requests

4 participants