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
F14815193: D9227.diff
Mon, Jan 27, 7:21 PM
Unknown Object (File)
Sun, Jan 26, 6:09 AM
Unknown Object (File)
Fri, Jan 24, 7:39 PM
Unknown Object (File)
Sat, Jan 18, 6:42 AM
Unknown Object (File)
Fri, Jan 17, 8:05 PM
Unknown Object (File)
Fri, Jan 17, 4:33 AM
Unknown Object (File)
Fri, Jan 17, 4:32 AM
Unknown Object (File)
Fri, Jan 17, 4:32 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
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