Page MenuHomePhabricator

Make Differential inline comment rendering more consistent and somewhat more modern
ClosedPublic

Authored by epriestley on Feb 21 2014, 7:36 PM.
Tags
None
Referenced Files
F19892410: D8294.id.diff
Sat, Mar 28, 1:20 AM
F19796987: D8294.id19722.diff
Feb 27 2026, 6:02 PM
F19791854: D8294.id19723.diff
Feb 26 2026, 12:02 AM
F19711545: D8294.id19723.diff
Feb 11 2026, 7:06 PM
F19694866: D8294.diff
Feb 10 2026, 11:20 AM
F19666836: D8294.diff
Feb 7 2026, 4:05 PM
F19506691: D8294.id19723.diff
Jan 9 2026, 4:50 PM
F19045626: D8294.diff
Nov 27 2025, 8:08 AM
Subscribers

Details

Summary

Ref T2222. Ref T1790. I partially modernized this recently, but bring it to the mail version too.

Test Plan

See screenshots.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/applications/differential/controller/DifferentialChangesetViewController.php
359

This true -> 1 stuff is for MySQL strict mode.

src/applications/differential/editor/DifferentialCommentEditor.php
662–663

Most of this juggling is to start unraveling our dependence on the old proxy/wrapper objects.

src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
43–45

Stuff is always passed in to this view in the correct order now, and we just respect it.