Closed Bug 1313850 Opened 3 years ago Closed 3 years ago

The 'Filter JSON' search box only works if the criteria is in lowercase, even if the matched attributes/values contain characters in uppercase

Categories

(DevTools :: JSON Viewer, defect)

49 Branch
defect
Not set

Tracking

(firefox52 fixed)

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: rick3162, Assigned: tephra)

Details

Attachments

(1 file)

With Firefox 49.0.2 x64 and Nightly 52 x64 in win10 x64.

For example: https://greasyfork.org/en/forum/discussions.json?script_author=1455
In that document, there are attributes like : "CountDiscussions" and values like "Markdown".
If you enter "CountDiscussions" and "MarkDown" in the search box, you'll get no results.
You'll only get results if you enter "countdiscussions" and "markdown" instead.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: nobody → eric
Comment on attachment 8809395 [details]
Bug 1313850 - Enable users to have capital letters in filter string

https://reviewboard.mozilla.org/r/92012/#review92310

Thanks for the patch!

R+

Honza
Attachment #8809395 - Flags: review?(odvarko) → review+
(In reply to Jan Honza Odvarko [:Honza] from comment #2)
> Comment on attachment 8809395 [details]
> Bug 1313850 - Enable users to have capital letters in filter string
> 
> https://reviewboard.mozilla.org/r/92012/#review92310
> 
> Thanks for the patch!
> 
> R+
> 
> Honza

Thanks for the review :)
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3b39fd1f2c80
Enable users to have capital letters in filter string r=Honza
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3b39fd1f2c80
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.