Page MenuHomePhabricator
Differential D18452

Update blame view in Diffusion
ClosedPublic
Actions

Authored by chad on Aug 22 2017, 4:13 AM.
Tags
None
Referenced Files
F15461660: D18452.id44333.diff
Tue, Apr 1, 10:02 AM
F15458499: D18452.diff
Mon, Mar 31, 4:06 AM
F15454946: D18452.id44330.diff
Sat, Mar 29, 9:25 PM
F15453909: D18452.diff
Sat, Mar 29, 3:01 PM
F15452421: D18452.id.diff
Sat, Mar 29, 4:40 AM
F15445250: D18452.diff
Thu, Mar 27, 1:08 PM
F15441678: D18452.diff
Wed, Mar 26, 7:19 PM
F15427502: D18452.diff
Sun, Mar 23, 2:43 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T12824: Redesign Diffusion
Commits
rPac91ab1ef919: Update blame view in Diffusion
Summary

Ref T12824, adds more information to the blame view, exposes date, commit summary, lighter colors.

Test Plan

Review many diffs with and without blame on.

Screen Shot 2017-08-21 at 9.07.23 PM.png (2×2 px, 791 KB)

Screen Shot 2017-08-21 at 9.07.31 PM.png (2×2 px, 709 KB)

Diff Detail

Repository
rP Phabricator
Branch
blame-colors (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18122
Build 24355: Run Core Tests
Build 24354: arc lint + arc unit

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
applications/
diffusion/
controller/
90 lines
webroot/
rsrc/
css/
application/
diffusion/
61 lines

Diff 44333

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
View Options

webroot/rsrc/css/application/diffusion/diffusion-source.css

Loading...
Phabricator · Built by Phacility