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
Unknown Object (File)
Thu, Dec 26, 6:24 AM
Unknown Object (File)
Tue, Dec 24, 5:13 PM
Unknown Object (File)
Thu, Dec 19, 6:49 PM
Unknown Object (File)
Thu, Dec 19, 11:02 AM
Unknown Object (File)
Fri, Dec 13, 11:25 PM
Unknown Object (File)
Fri, Dec 13, 10:02 AM
Unknown Object (File)
Thu, Dec 12, 7:30 PM
Unknown Object (File)
Sun, Dec 8, 3:29 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
Branch
inline-comments
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 101
Build 101: [Placeholder Plan] Wait for 30 Seconds

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 21079

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