Page MenuHomePhabricator

Restore editor behavior to Diffusion and support "\" shortcut
ClosedPublic

Authored by epriestley on Apr 19 2020, 3:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 10:36 PM
Unknown Object (File)
Mon, Dec 16, 4:52 AM
Unknown Object (File)
Mon, Dec 16, 4:52 AM
Unknown Object (File)
Mon, Dec 16, 4:52 AM
Unknown Object (File)
Sun, Dec 15, 3:37 AM
Unknown Object (File)
Sun, Dec 8, 3:39 PM
Unknown Object (File)
Fri, Dec 6, 7:22 AM
Unknown Object (File)
Nov 20 2024, 9:49 AM
Subscribers
None

Details

Summary

Ref T13515. This restores the "Open in Editor" behavior to Diffusion, and makes "\" work there.

The URI pattern is now sent as a structured template to the client, so the code will work properly if a file path contains "%l".

Test Plan
  • Clicked "Open in Editor" and pressed "\" in Diffusion when viewing a file.
  • Clicked a line, hit "\", got the file opened to that line.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Apr 19 2020, 4:41 PM
This revision was automatically updated to reflect the committed changes.