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
Unknown Object (File)
Sat, Jun 6, 1:38 PM
Unknown Object (File)
Mar 28 2026, 1:20 AM
Unknown Object (File)
Feb 27 2026, 6:02 PM
Unknown Object (File)
Feb 26 2026, 12:02 AM
Unknown Object (File)
Feb 11 2026, 7:06 PM
Unknown Object (File)
Feb 10 2026, 11:20 AM
Unknown Object (File)
Feb 7 2026, 4:05 PM
Unknown Object (File)
Jan 9 2026, 4:50 PM
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.