Closed Bug 1613882 Opened 6 years ago Closed 6 years ago

Use Accordion for the Params Panel

Categories

(DevTools :: Netmonitor, defect, P3)

Product:
DevTools
Component:
Netmonitor
Type:
defect

Tracking

(firefox75 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 75
Tracking Flags:
Tracking Status
firefox75 --- fixed

People

(Reporter: Honza, Assigned: bomsy)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1613882 - Moved ParamsPanel.js to request-details/ParamsPanel.js r=honza
47 bytes, text/x-phabricator-request
Details | Review

The Params panel should used Accordion component [1] to render its sections
The panel should also be moved into a new request-details directory.

Sections:

  • Form data
  • JSON preview
  • Post payload
  • Query String

[1] https://searchfox.org/mozilla-central/rev/3811b11b5773c1dccfe8228bfc7143b10a9a2a99/devtools/client/shared/components/Accordion.js#20

Honza

Bug 1613882 - Moved ParamsPanel.js to request-details/ParamsPanel.js r=honza

Bug 1613882 - Refactored params panel, added previews/SourcePreview, fix failing tests r=honza

Bug 1613882 - fix lint issues r=honza

Assignee: nobody → hmanilla
Status: NEW → ASSIGNED
Pushed by hmanilla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c614a62b4688 Moved ParamsPanel.js to request-details/ParamsPanel.js r=Honza

https://hg.mozilla.org/mozilla-central/rev/c614a62b4688

Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox75: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 75
Regressions: 1618511
Regressions: 1618600
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: