Page MenuHomePhabricator

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

Authored by avivey on May 20 2014, 11:00 PM.
Tags
None
Referenced Files
F18111494: D9227.diff
Tue, Aug 12, 11:05 AM
F18091667: D9227.id.diff
Thu, Aug 7, 12:05 AM
F18089768: D9227.id21998.diff
Wed, Aug 6, 12:22 PM
F18083879: D9227.diff
Tue, Aug 5, 12:10 PM
F18050875: D9227.id.diff
Sun, Aug 3, 9:21 PM
F17937649: D9227.id21913.diff
Jul 31 2025, 1:25 AM
F17857107: D9227.id21998.diff
Jul 27 2025, 6:07 PM
F17839244: D9227.id21913.diff
Jul 26 2025, 7:46 PM
Subscribers

Details

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 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.

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 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 updated this revision to Diff 21998.

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