-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Enable download-bubble feature by default #28562
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
Comments
Can we enable this now from security point of view? cc @diracdeltas @fmarier |
It's enabled on my Chrome Win11 but not on macOS. and our download page doesn't fit well with download bubble UI. (macOS) |
This is already being fixed, thankfully 👍 https://github.com/brave/brave-com-resources/issues/668 |
That sounds good to me. |
Verified
|
Brave | 1.50.90 Chromium: 111.0.5563.64 (Official Build) beta (x86_64) |
---|---|
Revision | c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995} |
OS | macOS Version 11.7.4 (Build 20G1120) |
Steps:
- installed
1.50.x
- launched Brave
- loaded
https://www.thinkbroadband.com/download
- context-clicked on
http://ipv4.download.thinkbroadband.com:8080/1GB.zip
- chose
Save Link As...
- clicked
Save
- confirmed download was initially blocked, due to being an
http://
resource on anhttps://
site - clicked on the blocked notification
- clicked on the
Continue download
button - confirmed download resumed
- confirmed download progress was reflected in the
Downloads
button - clicked on the above button
- confirmed double-click and context-menu options work as expected
example | example | example | example |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
example | example | example | example |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Also confirmed that both Enable download bubble...
flags are set to Default
, in brave://flags
Verification passed on
Brave | 1.50.93 Chromium: 111.0.5563.64 (Official Build) beta (64-bit) |
---|---|
Revision | c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995} |
OS | Ubuntu 18.04 LTS |
Steps:
- installed
1.50.x
- launched Brave
- loaded
https://www.thinkbroadband.com/download
- context-clicked on
http://ipv4.download.thinkbroadband.com:8080/1GB.zip
- chose
Save Link As...
- clicked
Save
- confirmed download was initially blocked, due to being an
http://
resource on anhttps://
site - clicked on the blocked notification
- clicked on the
Continue download
button - confirmed download resumed
- confirmed download progress was reflected in the
Downloads
button - clicked on the above button
- confirmed double-click and context-menu options work as expected
Verification
Case 1:
|
step 4 | step 5-6 | step 8-10 | step 11 | step 13 | step 14 | step 15 |
---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
brave://flags
Also confirmed that both Enable download bubble... flags are set to Default, in brave://flags
Case 2: New private window
- PASSED
- new profile
- launched Brave
- click Hamburger menu > New private window
- loaded
https://ubuntu.com/download/desktop/thank-you?version=22.04.2&architecture=amd64
- chose
Save Link As...
- clicked
Save
- confirmed download shelf no longer displayed
- confirmed download progress was reflected in the
Downloads
button - confirmed download progress was reflected in
brave://downloads
- confirmed download successfully completed and shown in Download icon in the toolbar and brave://downloads show downloaded file
- visited brave://downloads in Normal window
- confirmed file downloads are not shown
step 7 | step 9 | step 10 | step 11 | step 12-13 |
---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
Case 3: New private window with Tor
- PASSED
- new profile
- launched Brave
- click Hamburger menu > New private with Tor window
- loaded
https://www.thinkbroadband.com/download
- checked
Verify you are human
checkbox - context-clicked on
http://ipv4.download.thinkbroadband.com:8080/200GB.zip
- chose
Save Link As...
- clicked
Save
- confirmed download shelf no longer displayed
- confirmed download was initially blocked, due to being an
http://
resource on anhttps://
site - clicked on the blocked notification
- clicked on the
Continue download
button - confirmed download resumed
- confirmed download progress was reflected in the
Downloads
button - confirmed download progress was reflected in
brave://downloads
- confirmed context-menu options work as expected and successfully completed download and Download icon shown in in the toolbar and brave://downloads show downloaded file
- visited brave://downloads in Normal window
- confirmed filed downloads are not shown
step 4 | step 5 | step 6 | step 11 | step 12 | step 14 | step 15 | step 16 | step 18 |
---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case 4: Guest window
-PASSED
- new profile
- launched Brave
- click Hamburger menu > open guest window
- loaded
https://www.thinkbroadband.com/download
- context-clicked on
http://ipv4.download.thinkbroadband.com:8080/512MB.zip
- chose
Save Link As...
- clicked
Save
- confirmed download shelf no longer displayed
- confirmed download was initially blocked, due to being an
http://
resource on anhttps://
site - clicked on the blocked notification
- clicked on the
Continue download
button - confirmed download resumed
- confirmed download progress was reflected in the
Downloads
button - confirmed download progress was reflected in
brave://downloads
- confirmed context-menu options work as expected and successfully completed download and Download icon shown in in the toolbar and brave://downloads downloaded file
- visited brave://downloads in Normal window
- confirmed filed downloads are not shown
step 5 | step 7 | step 10 | step 11 | step 13 | step 14 | step 16 | step 17 |
---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case 5: Additional functionality in the download bubble - Pause
, Resume
, Cancel
, Retry
- PASSED
- new profile
- launched Brave
- loaded
https://ubuntu.com/download/desktop/thank-you?version=22.04.2&architecture=amd64
- chose
Save Link As...
- clicked
Save
- confirmed download shelf no longer displayed
- confirmed download progress was reflected in the
Downloads
button - clicked
Downloads
icon andPause
the download in the downloads dialog - confirmed download paused and shown
brave://downloads
- resumed download
- confirmed download resumed
- clicked
x
to cancel the download - confirmed download is cancelled and shown in the download bubble and
brave://downloads
- clicked
Retry
in brave://downloads` - confirmed download restarted and download progress was reflected in the
Downloads
button and inbrave://downloads
- confirmed download successfully completed and shown in Download icon in the toolbar and
brave://downloads
show downloaded file
step 4-5 | step 7 | step 8-9 | step 10-11 | step 12-13 | step 14 | step 15-16 | step 17 |
---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case 6: Upgrade - PASSED
- Install 1.49.120
- launch Brave
- loaded https://www.thinkbroadband.com/download
- context-clicked on http://ipv4.download.thinkbroadband.com:8080/1GB.zip
- chose
Save Link As...
- clicked
Save
- verified download message shown on the download shelf.
- clicked
Keep
- click
Exit
and close the browser - renamed the profile to Brave-Browser-Beta
- installed 1.50.101
- launched Brave
- opened brave://downloads and clicked
Retry
- clicked
Save
- confirmed download progress was reflected in the
Downloads
button - confirmed download progress was reflected in
brave://downloads
- confirmed context-menu options work as expected and successfully completed download and Download icon shown in in the toolbar and brave://downloads downloaded file
step 1 | step 4 | step 6 | step 8 | step 9 | step 13 | step 14 | step 15 | step 16 | step 17 |
---|---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Note: Download button styling issue: #29324
The |
this sounds like a bug. cc @mkarolin |
@ArkadiuszNiemiec, I wasn't able to reproduce. Is there a specific webpage you could point me to where you encountered this behavior? Thanks. |
I just use the link from top: http://ipv4.download.thinkbroadband.com:8080/1GB.zip Maybe there's something wrong on my setup, but I did not change anything other than update the Brave. I recorded it: |
@ArkadiuszNiemiec Thank you for the additional info and the video. I initially thought it wasn't asking with the option turned on, but the problem is it still asks in the private window when the option is toggled to off. Created a new issue for this: #29823 |
This option is very annoying, the previous way of downloading was much friendlier. |
@cyfra80 I agree, but the problem is that Google beinh Google has decided the faith of modern internet browsers yet again. Google changed the Chromium code base which is used by Brave (and a lot of other browsers). If I saw the Git commits in the Chromium repo correctly it shouldn't be too much work to add it back though (could be wrong as I don't know a lot about programming). See Chromium GIT for the mentioned file changes. |
Chromium has two features for download bubble
When it's enabled, user can see download status from bubble in toolbar instead of download shelf.
STR
cc @rebron
The text was updated successfully, but these errors were encountered: