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
F19504546: D21149.id.diff
Jan 9 2026, 5:19 AM
F19504526: D21149.id.diff
Jan 9 2026, 5:16 AM
F19083283: D21149.id50358.diff
Dec 2 2025, 12:33 PM
F18985443: D21149.id50347.diff
Nov 17 2025, 1:02 PM
F18852800: D21149.id.diff
Oct 31 2025, 12:02 PM
F18830423: D21149.id50347.diff
Oct 25 2025, 6:43 AM
F18829348: D21149.diff
Oct 24 2025, 9:19 PM
F18808044: D21149.id.diff
Oct 19 2025, 4:45 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
Branch
sticky12
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24165
Build 33285: Run Core Tests
Build 33284: arc lint + arc unit

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.