Diffusion Phabricator a5f62235537d

Show blame colors on all lines, instead of only the first affected line
a5f62235537d
Actions

Description

Show blame colors on all lines, instead of only the first affected line

Summary: Fixes T10226. I just made a mistake here when rewriting this recently.

Test Plan:

Screen Shot 2016-01-28 at 11.09.53 AM.png (1×1 px, 287 KB)

Reviewers: chad, avivey

Reviewed By: avivey

Maniphest Tasks: T10226

Differential Revision: https://secure.phabricator.com/D15131

Details

Provenance

epriestley	Authored on Jan 28 2016, 7:10 PM
epriestley	Pushed on Jan 28 2016, 7:17 PM

Reviewer

avivey

Differential Revision

D15131: Show blame colors on all lines, instead of only the first affected line

Parents

rP8f0d9c329566: Remove email prefixes from doorkeeper titles

Branches

Unknown

Tags

Unknown

Tasks

T10226: diffusion blame: Only first line number is highlighted

Build Status

Buildable 10373
Build 12654: Run Core Tests

Event Timeline

epriestley committed rPa5f62235537d: Show blame colors on all lines, instead of only the first affected line (authored by epriestley).Jan 28 2016, 7:17 PM

epriestley added a task: T10226: diffusion blame: Only first line number is highlighted.

Harbormaster completed building B10373: rPa5f62235537d: Show blame colors on all lines, instead of only the first affected line.Jan 28 2016, 7:17 PM

Changes (1)

Path

Size

src/

applications/

diffusion/

controller/

DiffusionBrowseController.php

rPa5f62235537d

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php