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 has started' toast notification sometimes appears when it shouldn't #6138

Closed
4 tasks done
subraizada3 opened this issue Apr 22, 2021 · 0 comments · Fixed by #6139
Closed
4 tasks done

'Download has started' toast notification sometimes appears when it shouldn't #6138

subraizada3 opened this issue Apr 22, 2021 · 0 comments · Fixed by #6139
Labels
bug Issue is related to a bug downloader Issue is related to the downloader

Comments

@subraizada3
Copy link
Contributor

Checklist

Steps to reproduce the bug

See the 9 second video below, look at toast notifications. Or the written version:

Open a video, click the download button which opens a download popup dialog box, and then press OK to start the download. At this point the popup closes, a toast notification saying 'Download has started' appears, and the download starts.

Click the download button again, which opens the download options popup. Click OK to start the download. At this point, it opens a second popup saying that either there is a download in progress with this filename, or that this video is already downloaded. This second popup asks whether you want to cancel the operation (in which case the download never starts), or overwrite the existing download / generate a unique filename. However, when you click OK in the first dialog, as it opens this second dialog it simultaneously creates the toast saying that 'Download has started' - when in fact a download has not started.

Actual behaviour

See above.

Expected behavior

This toast should only show if the user chooses to overwrite or generate a unique name in the second popup.

Screenshots/Screen recordings

video.mp4

Device info

  • Android version/Custom ROM version: LineageOS 18.1
  • Device model: Pixel 5

This functionality was added in feature request issue #5428 and implemented in PR #5519.

@subraizada3 subraizada3 added the bug Issue is related to a bug label Apr 22, 2021
@AudricV AudricV added the downloader Issue is related to the downloader label Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug downloader Issue is related to the downloader
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants