Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Findbar clear button isn't working #4965

Closed
cndouglas opened this issue Oct 19, 2016 · 6 comments · Fixed by #5040
Closed

Findbar clear button isn't working #4965

cndouglas opened this issue Oct 19, 2016 · 6 comments · Fixed by #5040

Comments

@cndouglas
Copy link

Did you search for similar issues before submitting this one?
Yes. Did not find anything.

Describe the issue you encountered:
Clicking on the findbar clear button doesn't do anything. This worked fine through 0.12.3 but stopped working in 0.12.4. 27bd13f probably caused this issue. cc: @diracdeltas

Expected behavior:
The clear button clears all the text in the findbar.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    macOS 10.12
  • Brave Version:
    0.12.4, 0.12.5, and 0.12.6 dev preview 1
  • Steps to reproduce:
    1. Open Brave and navigate to any website.
    2. Open the findbar with Ctrl/Cmd+F.
    3. Type some text in the findbar.
    4. Click the clear ("X") button.
  • Screenshot if needed:
    image
    The "X" button in the screenshot is not working.
  • Any related issues:
    None.
@luixxiul luixxiul added the bug label Oct 20, 2016
@bbondy bbondy added this to the 0.12.6dev milestone Oct 20, 2016
@diracdeltas
Copy link
Member

@jkup @bradleyrichter is this close button even necessary anymore, given that there's another close button on the findbar in the far right?

@cndouglas
Copy link
Author

@diracdeltas The button I'm talking about is the clear button, not the close button. This button is supposed to clear the find text so another search can be performed. The close button in the far right closes the entire findbar for when you are done searching.

@diracdeltas
Copy link
Member

@liunkae my bad, i guess i never use either of those buttons

@alexwykoff
Copy link
Contributor

moved to 0.12.7

@jkup
Copy link
Contributor

jkup commented Oct 21, 2016

@alexwykoff is it too late for this to be included in 12.6? The PR is up.

@cndouglas
Copy link
Author

Since we already have a fix for this bug, I also request that this be included in 0.12.6.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants