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)
Fri, Feb 14, 10:58 PM
Unknown Object (File)
Mon, Feb 10, 3:13 AM
Unknown Object (File)
Sun, Feb 9, 11:35 AM
Unknown Object (File)
Sun, Feb 9, 11:35 AM
Unknown Object (File)
Sun, Feb 9, 11:35 AM
Unknown Object (File)
Sun, Feb 9, 11:35 AM
Unknown Object (File)
Wed, Jan 29, 5:54 AM
Unknown Object (File)
Jan 22 2025, 1:31 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.