HomePhabricator
Diffusion Phabricator 8c4f5aba3339

Fix Back to HEAD link
8c4f5aba3339
Actions

Description

Fix Back to HEAD link

Summary: I missed an anchor tag here, adds it back

Test Plan: View blame, click a previous version of the file, click Back to HEAD link.

Reviewers: epriestley

Reviewed By: epriestley

Spies: Korvin

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

Details

Provenance
Chad Little <chad@phacility.com>Authored on Aug 22 2017, 1:47 AM
chadPushed on Aug 23 2017, 4:47 PM
Reviewer
epriestley
Differential Revision
D18451: Fix Back to HEAD link
Parents
rPac91ab1ef919: Update blame view in Diffusion
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18128
Build 24363: Run Core Tests

Event Timeline

chad committed rP8c4f5aba3339: Fix Back to HEAD link (authored by chad).Aug 23 2017, 4:47 PM
Harbormaster completed building B18128: rP8c4f5aba3339: Fix Back to HEAD link.Aug 23 2017, 4:49 PM
ftdysa added a subscriber: ftdysa.Aug 23 2017, 5:22 PM

I think that the Last Change button may also be missing an anchor tag. Clicking it on this page has no effect https://secure.phabricator.com/source/phabricator/browse/master/.arcunit but if I fiddle with the HTML and change the button to a then it works as expected.

chad added a comment.Aug 23 2017, 5:24 PM

thanks, i'll check it out.

Changes (1)

PathSize
src/
applications/
diffusion/
controller/

rP8c4f5aba3339

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
Phabricator ยท Built by Phacility