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

Delete button doesn't appear on brave://history when using the search box #7398

Closed
onmyouji opened this issue Dec 15, 2019 · 10 comments · Fixed by brave/brave-core#8910
Closed
Labels
about-pages/rebrand Rebranding work on top of Chrome's Polymer-powered pages bug feature/history feature/user-interface All UI related OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-macOS QA/Yes release-notes/include

Comments

@onmyouji
Copy link

onmyouji commented Dec 15, 2019

Description

Normally when you check the box next to an item on brave://history, the 'Delete' button will appear at the top right.

Image 001

But when you search something specific using the search bar, the 'Delete' button doesn't appear when you check the box. I initially thought that I have to delete the checked items one by one. But turns out I can still delete multiple items by using 'Delete' key on my keyboard. Still this is not really good in terms of UI and can be confusing for users.

Image 002

For comparison, on Chrome the 'Delete' button still appears when you use the search bar.

Image 003

Steps to Reproduce

  1. Open History (Ctrl+H) or type brave://history
  2. Click the Search button at top right, then type your search query. It will display the search results.
  3. Check the box next to an item. The 'Delete' button doesn't appear.

Actual result:

The 'Delete' button doesn't appear.

Expected result:

The 'Delete' button should appear.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave: 1.1.21 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision: 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS: Windows 10 OS Version 1809 (Build 17763.864)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
    Yes
  • Can you reproduce this issue with the beta channel?
    Haven't tested
  • Can you reproduce this issue with the dev channel?
    Haven't tested
  • Can you reproduce this issue with the nightly channel?
    Haven't tested

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
    No
  • Does the issue resolve itself when disabling Brave Rewards?
    No
  • Is the issue reproducible on the latest version of Chrome?
    No
@bsclifton bsclifton added about-pages/rebrand Rebranding work on top of Chrome's Polymer-powered pages bug feature/history labels Dec 19, 2019
@bsclifton
Copy link
Member

cc: @petemill

@colinhemphill
Copy link

Just encountered this one and had to go to inspect mode to access the Cancel and Delete options for my selections. Looks like the cr-toolbar-selection-overlay should take index priority over the cr-toolbar #main-toolbar.

@bsclifton
Copy link
Member

bsclifton commented Mar 30, 2020

I believe this is happening because the br_toolbar Polymer component is on top of the Cancel and Delete buttons (and the UI showing how many items matched search)

Labeled as P3 since this is broken functionality

@bsclifton bsclifton added feature/user-interface All UI related priority/P3 The next thing for us to work on. It'll ride the trains. labels Mar 30, 2020
@mhidro
Copy link

mhidro commented Jul 20, 2020

Sup. Is this being worked on?

@KarlisZ
Copy link

KarlisZ commented May 12, 2021

It is now may 2021. This can't be that difficult. please.

@stephendonner
Copy link

Verified FIXED using

Brave 1.28.31 Chromium: 91.0.4472.124 (Official Build) nightly (x86_64)
Revision 7345a6d1bfcaff81162a957e9b7d52649fe2ac38-refs/branch-heads/4472_114@{#6}
OS macOS Version 11.4 (Build 20F71)

Steps:

  1. clean profile
  2. launched Brave
  3. loaded search.brave.com and searched for dog
  4. clicked History -> Show Full History (brave://history/)
  5. clicked on the magnifying glass icon
  6. typed dog
  7. clicked on the [ ] checkbox to the left of the search result
  8. in the pop-up UI at the top, clicked on Delete
  9. confirmed I got a prompt to remove selected items
  10. clicked on Remove and confirmed history result was deleted
  11. also clicked on Cancel and the X icon while on step 8, and confirmed I was returned to the history-search view, with term in-place
example example example example
Screen Shot 2021-06-25 at 9 43 25 AM Screen Shot 2021-06-25 at 9 43 34 AM Screen Shot 2021-06-25 at 9 54 57 AM Screen Shot 2021-06-25 at 9 55 29 AM

@mhidro
Copy link

mhidro commented Jul 16, 2021

When can we expect this in stable? Also on Windows.

@stephendonner
Copy link

When can we expect this in stable? Also on Windows.

Per https://github.com/brave/brave-browser/wiki/Brave-Release-Schedule, August 10th.

@rebron rebron changed the title 'Delete' button doesn't appear on brave://history when using the search bar at the top 'Delete' button doesn't appear on brave://history when using the search box Aug 9, 2021
@rebron rebron changed the title 'Delete' button doesn't appear on brave://history when using the search box Delete button doesn't appear on brave://history when using the search box Aug 9, 2021
@levicki
Copy link

levicki commented May 27, 2024

@rebron Sorry for slight offtopic, is there a way to bulk delete site history which doesn't involve doing that by searching for site name, pressing Ctrl+A, clicking Delete, clicking Remove, waiting 5-10 seconds for 150 entries to be deleted and for next 150 entries to be shown, repeat until done?

Is this the best we can do in 2024?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
about-pages/rebrand Rebranding work on top of Chrome's Polymer-powered pages bug feature/history feature/user-interface All UI related OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-macOS QA/Yes release-notes/include
Projects
None yet
8 participants