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

Repository
rP Phabricator
Branch
dxiorder
Lint
Lint Passed
Unit
Tests Passed

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.