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

Search bar in History blocks buttons in UI #15572

Open
Brave-Matt opened this issue Apr 29, 2021 · 3 comments
Open

Search bar in History blocks buttons in UI #15572

Brave-Matt opened this issue Apr 29, 2021 · 3 comments
Assignees
Labels
feature/settings needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop priority/P5 Not scheduled. Don't anticipate work on this any time soon.

Comments

@Brave-Matt
Copy link

Description

When you search your browsing history in Settings --> History, the search bar appears and overlays the Cancel and Delete buttons you would use after selecting sites:

Without search:
image

With search:
image

Steps to Reproduce

  1. Visit Settings --> History
  2. Search using the search bar at the top
  3. Observe buttons being hidden

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.24.77 Chromium: 90.0.4430.85 (Official Build) dev (x86_64)
Revision 5bc145d831c180d9ff94f29a0d7a2e1cbd30ef36-refs/branch-heads/4430@{#1311}
OS macOS Version 11.2.3 (Build 20D91)

Note that this also happens on Windows systems.

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Miscellaneous Information:

User report:
https://community.brave.com/t/a-bug-in-ui-in-browser-history-page/240494

@rebron
Copy link
Collaborator

rebron commented May 3, 2021

Working as designed with the x button for closing the search. So I'm not sure there's a need for cancel and delete buttons for search whereas I see the need for selection.
cc: @petemill @karenkliu

@rebron rebron self-assigned this May 3, 2021
@rebron rebron added needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. priority/P5 Not scheduled. Don't anticipate work on this any time soon. labels May 3, 2021
@karenkliu
Copy link

Yeah, this is actually how Chrome works. You can't search history after selecting an item in history (in Gmail, can you still use the search function if something is selected, but searching mail will get rid of your selection and instead show the search results).
In our case, it looks funky because we have other stuff in the top nav that Chrome doesn't have, which also gets hidden if the user is engaging in searching history.

I propose leaving as is for now, but to make it less confusing in the future we might want to take the page-specific search icon out of the top nav - it seems that some people don't realize it's searching only the page and not ALL the content through Settings/Brave Rewards/History/Bookmarks/Downloads/Crypto Wallets. Also, some pages like Crypto Wallets don't require a page-search function.

Dropping the search function down to the page level might look something like this:
image

Of course, we'll have to think of the UI treatment for each the pages (Settings/Brave Rewards/History/Bookmarks/Downloads/Crypto Wallets) if we go in this direction.

@eljuno
Copy link

eljuno commented May 12, 2021

Duplicate of #7398 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/settings needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop priority/P5 Not scheduled. Don't anticipate work on this any time soon.
Projects
Status: P5 backlog
Development

No branches or pull requests

4 participants