Highlight React events with the same wakeable ID

taneliang commented 4 years ago

110 [Resize content][1/n] Break horizontal scrolling area into final-ish hierarchy
107 [Resize content][2/n] Implement ResizableSplitView
108 [Resize content][3/n] Sync horizontal pan and zoom states
109 Implement User Timing marks view
#111 Highlight React events with the same wakeable ID

Summary

Resolves #44.

Intended to highlight all related React events. However, it looks like separate promises can have the same wakeable ID, which is a bit strange.

TODO

Ensure that we are correctly understanding wakeable IDs, i.e. that a unique resource/promise is a unique wakeable.
Check if unstable_createResource caches promises.

Test Plan

yarn lint
yarn flow: no errors in changed code
yarn start

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/mlh-fellowship/scheduling-profiler-prototype/qxzqo7mwz ✅ Preview: https://scheduling-profiler-prototype-git-sttack-pariatur-dolorem-illum.mlh-fellowship.vercel.app

taneliang commented 4 years ago

Rebased on master.

We may want to use an intermediate highlight color for the related highlights in the future, so that it doesn't look like a bug in our highlighting logic

MLH-Fellowship / scheduling-profiler-prototype