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
F13070015: D8294.diff
Mon, Apr 22, 8:55 AM
Unknown Object (File)
Sat, Apr 20, 6:29 PM
Unknown Object (File)
Mon, Apr 8, 8:40 AM
Unknown Object (File)
Thu, Apr 4, 3:38 PM
Unknown Object (File)
Jan 26 2024, 7:54 AM
Unknown Object (File)
Jan 24 2024, 9:14 PM
Unknown Object (File)
Nov 25 2023, 7:27 AM
Unknown Object (File)
Nov 23 2023, 12:08 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.