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

"match case" doesn't work under the findbar #14019

Closed
kjozwiak opened this issue May 3, 2018 · 2 comments
Closed

"match case" doesn't work under the findbar #14019

kjozwiak opened this issue May 3, 2018 · 2 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented May 3, 2018

Description

When searching for words via the findbar, match case doesn't work and will highlight the matching words even though the case doesn't match.

Steps to Reproduce

  1. launch 0.22.702 e4a853d
  2. visit https://en.wikipedia.org/wiki/World_War_II
  3. open the findbar via CMD + F or CTRL + F
  4. Search for GeRmAnY and click enable Match Case
  5. press "enterand you'll notice that Brave keeps looking for instances ofGermanyeven though the text field hasGeRmAnY` as it's search term

Actual result:

matchcase

Expected result:

Using Match Case under the findbar should work as expected and correctly look for the case that's added into the text field.

Reproduces how often:

100% reproducible using the above STR.

Brave Version

about:brave info:

Brave: 0.22.702 
V8: 6.6.346.26 
rev: e4a853d9990c62328a8a4320fec21f58c9cdc0c1 
Muon: 6.0.7 
OS Release: 4.13.0-39-generic 
Update Channel: Beta 
OS Architecture: x64 
OS Platform: Linux 
Node.js: 7.9.0 
Brave Sync: v1.4.2 
libchromiumcontent: 66.0.3359.139

Reproducible on current live release:

Currently not reproducible under 0.22.669 which is the latest released version.

Additional Information

@kjozwiak kjozwiak added this to the 0.22.x Release 3 (Beta channel) milestone May 3, 2018
@petemill
Copy link
Member

petemill commented May 7, 2018

Should be fixed in next build. Was caused by a mispelling in some refactored code.

@btlechowski
Copy link
Contributor

btlechowski commented May 8, 2018

Verified on Ubuntu 17.10 x64

  • 0.22.703 903b8d0
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.8

Verified on Windows x64

  • 0.22.703 903b8d0
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.8

Verified with macOS 10.12.6 using

  • 0.22.703 903b8d0
  • muon 6.0.8
  • libchromiumcontent 66.0.3359.139

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

No branches or pull requests

6 participants