Page MenuHomePhabricator
Differential D8885

Update inline-comment styles in Differential
ClosedPublic
Actions

Authored by chad on Apr 28 2014, 4:45 PM.
Tags
None
Referenced Files
F18940463: D8885.id.diff
Tue, Nov 11, 8:29 AM
F18934502: D8885.diff
Nov 10 2025, 3:42 PM
F18830766: D8885.diff
Oct 25 2025, 8:39 AM
F18770966: D8885.diff
Oct 8 2025, 4:35 PM
F18624702: D8885.id.diff
Sep 15 2025, 8:50 PM
F18624700: D8885.id.diff
Sep 15 2025, 8:49 PM
F18607901: D8885.diff
Sep 14 2025, 12:20 AM
F18598803: D8885.diff
Sep 13 2025, 5:54 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rPa46ead798007: Update inline-comment styles in Differential
Summary

Moderize Inline Comment Display

  • Use standard colors
  • Better display with/without comment
  • OMG Icons
Test Plan

Screen_Shot_2014-04-28_at_9.42.52_AM.png (632×541 px, 46 KB)

Test with and without main comment, test with many for few comments on 1-3 files.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chad updated this revision to Diff 21078.Apr 28 2014, 4:45 PM
chad retitled this revision from to Update inline-comment styles in Differential.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptApr 28 2014, 4:45 PM
chad edited the test plan for this revision. (Show Details)Apr 28 2014, 4:46 PM
Harbormaster completed remote builds in B100: Diff 21078.Apr 28 2014, 4:46 PM
chad updated this revision to Diff 21079.Apr 28 2014, 4:48 PM
  • fix anchor width back to 100
Harbormaster completed remote builds in B101: Diff 21079.Apr 28 2014, 4:49 PM
btrahan accepted this revision.Apr 28 2014, 10:02 PM
btrahan edited edge metadata.

Haha, love the test data.

This revision is now accepted and ready to land.Apr 28 2014, 10:02 PM
chad closed this revision.Apr 29 2014, 2:43 AM
chad updated this revision to Diff 21091.

Closed by commit rPa46ead798007 (authored by @chad).

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
infrastructure/
diff/
view/
14 lines
webroot/
rsrc/
css/
application/
diff/
31 lines

Diff 21091

View Options

resources/celerity/map.php

Loading...
View Options

src/infrastructure/diff/view/PhabricatorInlineSummaryView.php

Loading...
View Options

webroot/rsrc/css/application/diff/inline-comment-summary.css

Loading...
Phabricator · Built by Phacility