Details
- Log the list of members at the end of `createPendingSidebar
- Tested in combination with the following diffs, where I show screenshots of a pending sidebar with the @-mentions bolded
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
lib/shared/thread-utils.js | ||
---|---|---|
503–519 ↗ | (On Diff #22682) | Note that this function (baseCreatePendingSidebar) gets called from two places: createPendingSidebar and createUnresolvedPendingSidebar. In the former case we are able to do ENS resolution really easily, so I initially implemented this code so that ENS names were resolved in the former case but not the latter. But then I realized that bolding text can change its height, and we need the height calculation to be consistent between these two functions in order for the sidebar animation to look correct. To do this right we'd probably want to trigger remeasurement of a message once ENS names come in, but doing this in a performant way will probably require adding a new provider component upstream of MessageListContainer that can handle the ENS querying and avoid repeated remeasures of the same (unchanged) component. We'll also need to think about the measureKey, which needs to include a list of mentionableNames somehow in order for the height-measuring code to realize the height may be different. Since the scope there is pretty high, and since we don't support @-mentioning ENS names yet anyways, I've decided to defer that work to ENG-1969. |
lib/shared/thread-utils.js | ||
---|---|---|
503–519 ↗ | (On Diff #22682) | Thanks for the thorough explanation + linking the Linear issue |