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

	Unknown Object (File)
	Wed, Jan 1, 9:30 PM

	Unknown Object (File)
	Thu, Dec 12, 12:40 AM

	Unknown Object (File)
	Nov 28 2024, 1:41 AM

	Unknown Object (File)
	Nov 26 2024, 12:50 AM

	Unknown Object (File)
	Nov 23 2024, 10:20 PM

	Unknown Object (File)
	Nov 19 2024, 1:47 AM

	Unknown Object (File)
	Oct 2 2024, 6:32 AM

	Unknown Object (File)
	Sep 16 2024, 10:57 AM

View All 73 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