Details
Details
Before:
After:
You can observe that we don't render anything nested within InnerTextMessage
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- arcpatch-D9055 (branched from master)
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
As of this diff we've brought the "second render" of MessageList from ~100ms to ~40ms. The ThreadList optimizations should be 4-5x as impactful.
native/chat/inner-text-message.react.js | ||
---|---|---|
130–133 ↗ | (On Diff #30669) | Probably can be defined outside the component |
native/chat/inner-text-message.react.js | ||
---|---|---|
130–133 ↗ | (On Diff #30669) | I think the cost of this is pretty low and would prefer to keep the styles "near" the JSX |