Closed Bug 1217131 Opened 9 years ago Closed 1 year ago

Filtering should show complete objects that match

Categories

(DevTools :: JSON Viewer, defect, P3)

defect

Tracking

(firefox44 affected)

RESOLVED DUPLICATE of bug 1358192
Tracking Status
firefox44 --- affected

People

(Reporter: canuckistani, Unassigned)

References

Details

(Whiteboard: [btpp-backlog])

Attachments

(3 files)

STR: 

1. open this url: http://live-menu.staugustinesvancouver.com/taps.json
2. filter on 'IPA'

Expected: I should see the entire object that contains property values that match 'IPA'

Actual: the objects themselves are filtered so you can't see the rest of the property names or values
I tested this based on your steps 1 and 2. I used the filter option on the browser and it showed the entire JSON object, wrapped of course and highlights all places marked IPA. Am I doing the test properly?
Flags: needinfo?(jgriffiths)
Attached image json-viewer-filter.gif
I've attached a gif of what I see in current Nightly. Notice when the filter is applied I only see properties in the 'Ninkasi IPA' object that contain 'IPA', but when I remove the filter completely I see the full object.
Flags: needinfo?(jgriffiths)
I totally agree on this. For example, on the following json, when searching for "address" I'd like to see the whole address object (with all its fields, "provinceCode", "macroregion", ...); it would also be very helpful if it expanded the json automatically (as of 45.0a2 (2016-01-24) the json is not expanded when a search is performed).

{
  "_type": "person",
  "_id": "e56fdc1eeef8",
  "_score": 16.13567,
  "_source": {
    "taxID": "***",
    "address": {
      "provinceCode": "TV",
      "macroregion": "Nord-est",
      "postcode": "31026",
      "municipality": "Sarmede",
      "province": "Treviso",
      "toponym": "***",
      "streetName": "***",
      "streetNumber": "***",
      "fullAddress": "***, 31026, Sarmede (TV)",
      "region": "Veneto",
      "state": "Italia"
    },
    "name": "***",
    "gender": "M",
    "birthPlace": {
      "provinceCode": "TV",
      "macroregion": "Nord-est",
      "municipality": "**",
      "province": "Treviso",
      "region": "Veneto",
      "state": "Italia"
    },
    "surname": "***",
    "birthDate": "1962-01-23"
  }
}
Attached patch 1217131.patchSplinter Review
Just took a rough stab at this by passing down the search match status as a bool.  It works but needs some help.

The search highlight needs to not highlight parents e.g. not(.hasChildren)
Also the search highlight color is purposely hideous
And we should open the objects by default, this can be done with member.open = true but I saw some perf issues when doing it.
screenshot of the patch working
Priority: -- → P3
Whiteboard: [btpp-backlog]
Product: Firefox → DevTools
Severity: normal → S3
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: