Page MenuHomePhabricator
Differential D15131

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

Authored by epriestley on Jan 28 2016, 7:10 PM.
Tags
None
Referenced Files
F15407331: D15131.id36533.diff
Tue, Mar 18, 4:59 PM
F15392512: D15131.diff
Sat, Mar 15, 3:38 PM
F15389480: D15131.id36533.diff
Sat, Mar 15, 5:19 AM
F15334034: D15131.diff
Sat, Mar 8, 6:22 AM
F15333995: D15131.diff
Sat, Mar 8, 6:08 AM
Unknown Object (File)
Mon, Feb 24, 7:55 AM
Unknown Object (File)
Jan 30 2025, 4:03 AM
Unknown Object (File)
Jan 25 2025, 4:09 PM
View All Files
Subscribers
None

Details

Reviewers
avivey
chad
Maniphest Tasks
T10226: diffusion blame: Only first line number is highlighted
Commits
Restricted Diffusion Commit
rPa5f62235537d: 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)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36533.Jan 28 2016, 7:10 PM
epriestley retitled this revision from to Show blame colors on all lines, instead of only the first affected line.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10226: diffusion blame: Only first line number is highlighted.
avivey accepted this revision.Jan 28 2016, 7:17 PM
avivey added a reviewer: avivey.
This revision is now accepted and ready to land.Jan 28 2016, 7:17 PM
Closed by commit rPa5f62235537d: Show blame colors on all lines, instead of only the first affected line (authored by epriestley, committed by epriestley). · Explain WhyJan 28 2016, 7:17 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
6 lines

Diff 36534

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
Phabricator · Built by Phacility