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
Unknown Object (File)
Jun 15 2025, 6:47 AM
Unknown Object (File)
Jun 5 2025, 8:43 AM
Unknown Object (File)
Jun 1 2025, 4:54 PM
Unknown Object (File)
May 30 2025, 10:41 AM
Unknown Object (File)
May 29 2025, 10:16 PM
Unknown Object (File)
May 28 2025, 2:52 PM
Unknown Object (File)
May 9 2025, 5:14 AM
Unknown Object (File)
May 9 2025, 12:38 AM
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
Lint
Lint Skipped
Unit
Tests Skipped

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
resources/
celerity/
18 lines
src/
applications/
diffusion/
controller/
3 lines
webroot/
rsrc/
js/
core/
5 lines

Diff 21998

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseFileController.php

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility