Closed Bug 1761510 Opened 1 year ago Closed 1 year ago

Minimize number of rendered elements in the react app


(DevTools :: Console, task)



(firefox100 fixed)

100 Branch
Tracking Status
firefox100 --- fixed


(Reporter: nchevobbe, Assigned: nchevobbe)



(Whiteboard: dt-console-perf-2022)


(3 files)

Looking at the generated html, it looks like we could skip rendering some items.
Hopefully this will help a bit with reflow/styling/overall react perf when we have virtualization and the user scrolls

This simplifies the markup a bit, and hopefully could contribute to improve
layout and styling performance in the console.

we directly use what's returned from the Frame component. We allow to pass
an extra className prop to it so we can still add the message-location class
that is used in the console.

Depends on D142096

We move the data-indent attribute to the message element so tests can still
query it.

Depends on D142097

Pushed by
[devtools] Remove frame-link-source-inner element in Frame component. r=bomsy.
[devtools] Remove extra element for console message location. r=bomsy.
[devtools] Don't render indent element when it's not needed. r=bomsy.
Regressions: 1804384
You need to log in before you can comment on or make changes to this bug.