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
F15490351: D8885.id21078.diff
Fri, Apr 11, 4:23 PM
F15486134: D8885.diff
Thu, Apr 10, 5:14 AM
F15485505: D8885.id21079.diff
Thu, Apr 10, 12:14 AM
F15477491: D8885.diff
Mon, Apr 7, 3:22 PM
F15476107: D8885.id21091.diff
Mon, Apr 7, 4:04 AM
F15471863: D8885.id.diff
Sat, Apr 5, 8:39 AM
F15461215: D8885.id21078.diff
Tue, Apr 1, 6:16 AM
F15432207: D8885.diff
Mar 24 2025, 4:46 PM
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