Page MenuHomePhabricator

Fix an issue with 1up diff block rendering for added or removed blocks
ClosedPublic

Authored by epriestley on Nov 8 2019, 3:35 PM.
Tags
None
Referenced Files
F15453672: D20897.id49810.diff
Sat, Mar 29, 1:30 PM
F15450563: D20897.id.diff
Fri, Mar 28, 4:28 PM
F15443729: D20897.diff
Thu, Mar 27, 5:52 AM
F15412589: D20897.diff
Mar 19 2025, 1:51 PM
Unknown Object (File)
Feb 26 2025, 4:01 AM
Unknown Object (File)
Feb 21 2025, 1:21 PM
Unknown Object (File)
Feb 9 2025, 8:25 AM
Unknown Object (File)
Feb 9 2025, 8:25 AM
Subscribers

Details

Summary

Ref T13425. When a change adds or removes a block (vs adding or removing a document, or changing a block), we currently try to render null as cell content in the unified view. Make this check broader to catch both "no opposite document" and "no opposite cell".

Test Plan

Screen Shot 2019-11-08 at 7.36.31 AM.png (492×1 px, 76 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable