Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Download prompts can appear after switching tabs #38551

Closed
2 of 5 tasks
kylehickinson opened this issue May 24, 2024 · 4 comments · Fixed by brave/brave-core#23899
Closed
2 of 5 tasks

Download prompts can appear after switching tabs #38551

kylehickinson opened this issue May 24, 2024 · 4 comments · Fixed by brave/brave-core#23899
Assignees
Labels
bug OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - iPad QA Pass - iPhone QA/Test-All-Platforms QA/Yes release-notes/include security

Comments

@kylehickinson
Copy link
Collaborator

Description

Tapping on a link that would prompt the user to download a file can appear even after switching tabs

Steps to reproduce

  1. Find a page to download something (e.g. 1Password download page)
  2. Click to download or paste download URL
  3. Quickly tap the + button in the toolbar to add a new tab

Actual result

image

Expected result

The download prompt should be tied to the tab itself and not appear when the tab has been backgrounded

Reproduces how often

Easily reproduced

Brave version

1.66 but likely an issue for a long time

Device/iOS version

iOS 17.4 (iPhone), but will be a problem on all devices/versions

Affected browser versions

  • latest AppStore
  • latest TestFlight
  • previous TestFlight

Reproducibility

  • with Brave Shields disabled
  • in the latest version of mobile Safari

Miscellaneous information

No response

@kylehickinson kylehickinson added bug QA/Yes OS/iOS Fixes related to iOS browser functionality labels May 24, 2024
@stoletheminerals stoletheminerals added security priority/P2 A bad problem. We might uplift this to the next planned release. labels May 24, 2024
@brave-builds brave-builds added this to the 1.68.x - Nightly milestone May 30, 2024
@iccub
Copy link

iccub commented Jun 4, 2024

@stoletheminerals hi do you want this uplifted or can it sit in release version 1.68?

@stoletheminerals
Copy link

@iccub would be good to get it uplifted

@kjozwiak
Copy link
Member

kjozwiak commented Jun 5, 2024

The above requires 1.66.120 or higher for 1.66.x verification 👍

@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jun 5, 2024
@hffvld
Copy link
Contributor

hffvld commented Jun 5, 2024

Verified on iPhone 14 and iPad Air using version(s):

Device/OS: 
- iPhone 14 / iOS 17.5
- iPad Air / iPadOS 16.7.2
Brave build: 1.66.1 (120)
BraveCore: 1.66.120 (125.0.6422.165)

STEPS:

  1. Follow the steps from Download prompts can appear after switching tabs #38551 (comment)
  2. Verify

ACTUAL RESULTS:

  • Verified that the download pop-up message is not shown on the NTP
  • Verified that it's still possible to download a file from the correct page

Reproduced in 1.66 (113)

2024-06-05_13-09-53.mp4

Verified In 1.66.1 (120) on iPhone

2024-06-05_13-31-10.mp4

Verified In 1.66.1 (120) on iPad

2024-06-05_15-23-13.mp4

@hffvld hffvld added QA Pass - iPhone QA Pass - iPad and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - iPad QA Pass - iPhone QA/Test-All-Platforms QA/Yes release-notes/include security
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants