Support "control" and "return/enter" in the remarkup rule for keystrokes
ClosedPublic
Actions

Authored by epriestley on Jan 12 2021, 3:43 AM.

Tags

None

Referenced Files

	F13157318: D21508.diff
	Sun, May 5, 10:48 AM

	Unknown Object (File)
	Sat, May 4, 9:25 AM

	Unknown Object (File)
	Fri, May 3, 5:08 AM

	Unknown Object (File)
	Sat, Apr 27, 4:41 PM

	Unknown Object (File)
	Thu, Apr 25, 12:46 AM

	Unknown Object (File)
	Apr 5 2024, 5:26 PM

	Unknown Object (File)
	Mar 28 2024, 3:26 PM

	Unknown Object (File)
	Feb 16 2024, 4:51 AM

View All 39 Files

Subscribers

None

Details

Reviewers: None
Commits: rPc63c2aadef3a: Support "control" and "return/enter" in the remarkup rule for keystrokes

Summary

These characters are missing support in {key ...} but are reasonable to include.

Test Plan

Screen Shot 2021-01-11 at 7.41.17 PM.png (459×436 px, 23 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 12 2021, 3:43 AM

Harbormaster completed remote builds in B25006: Diff 51193.Jan 12 2021, 3:44 AM

epriestley requested review of this revision.Jan 12 2021, 3:44 AM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 12 2021, 3:47 AM

Closed by commit rPc63c2aadef3a: Support "control" and "return/enter" in the remarkup rule for keystrokes (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPc63c2aadef3a: Support "control" and "return/enter" in the remarkup rule for keystrokes.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

markup/

rule/

PhabricatorKeyboardRemarkupRule.php

16 lines

Diff 51194

View Options

src/infrastructure/markup/rule/PhabricatorKeyboardRemarkupRule.php

Support "control" and "return/enter" in the remarkup rule for keystrokesClosedPublicActions