Differential D10282 Authored by ginsu on Dec 8 2023, 9:55 PM.
Tags None Referenced Files
Details Summary As I was working to visaully cleanup the thread notifications modal, I found the rendering logic for this modal pretty confusing/hard to read. Broke down the different parts of this rendering logic + memoized them Depends on D10279 Test Plan Confirmed that there were no regressions with the thread notificaitons modal
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptDec 8 2023, 9:55 PM2023-12-08 21:55:57 (UTC+0) Harbormaster completed remote builds in B24928: Diff 34462.Dec 8 2023, 10:18 PM2023-12-08 22:18:24 (UTC+0) This revision is now accepted and ready to land.Dec 11 2023, 1:25 AM2023-12-11 01:25:54 (UTC+0) ginsu added a child revision: D10315: [web] cleanup thread notifications modal.Dec 12 2023, 8:41 PM2023-12-12 20:41:34 (UTC+0) Harbormaster completed remote builds in B26867: Diff 37143.Feb 15 2024, 8:43 AM2024-02-15 08:43:58 (UTC+0) Closed by commit rCOMM0106682a76f7: [web] cleanup thread notifications modal rendering logic. · Explain WhyFeb 15 2024, 8:44 AM2024-02-15 08:44:03 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 37143 |