Page MenuHomePhabricator
Differential D9227

Diffusion browser: Update editor link when clicking on a line
ClosedPublic
Actions

Authored by avivey on May 20 2014, 11:00 PM.
Tags
None
Referenced Files
F19004547: D9227.diff
Nov 21 2025, 11:59 AM
F18787213: D9227.id.diff
Oct 14 2025, 10:36 PM
F18784029: D9227.diff
Oct 13 2025, 11:41 AM
F18734894: D9227.id.diff
Sep 30 2025, 11:05 PM
F18595797: D9227.id.diff
Sep 12 2025, 10:54 PM
F18584082: D9227.diff
Sep 11 2025, 11:52 AM
F18470543: D9227.diff
Sep 2 2025, 5:59 PM
F18468220: D9227.diff
Sep 2 2025, 2:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP9bba4cda2fe5: Diffusion browser: Update editor link when clicking on a line
Summary

Highlighing and URL are fixed on click - now the edit button too.

Test Plan

click on lines with and without value in "Editr Link" (And without %l in it).

Diff Detail

Repository
rP Phabricator
Branch
fixeditlink
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 612
Build 612: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

avivey updated this revision to Diff 21913.May 20 2014, 11:00 PM
avivey retitled this revision from to Diffusion browser: Update editor link when clicking on a line.
avivey updated this object.
avivey edited the test plan for this revision. (Show Details)
avivey added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 20 2014, 11:00 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B612: Diff 21913.May 20 2014, 11:01 PM
avivey added a comment.EditedMay 20 2014, 11:02 PM

This actually doesn't work if the Editor Link has %25l literal in it. I could "fix" it by adding '%25' => '%%325' conversion in loadEditorLink(), but it would still be wrong for %%25l...

epriestley accepted this revision.May 22 2014, 10:33 PM
epriestley edited edge metadata.

e h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h h . . . okay

This revision is now accepted and ready to land.May 22 2014, 10:33 PM
epriestley closed this revision.May 22 2014, 10:33 PM
epriestley updated this revision to Diff 21998.

Closed by commit rP9bba4cda2fe5 (authored by @avivey, committed by @epriestley).

avivey added a comment.May 22 2014, 10:39 PM

Thanks!

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
3 lines
webroot/
rsrc/
js/
core/
5 lines

Diff 21913

View Options

src/applications/diffusion/controller/DiffusionBrowseFileController.php

Loading...
View Options

webroot/rsrc/js/core/behavior-line-linker.js

Loading...
Phabricator · Built by Phacility