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

	F14918350: D21508.diff
	Thu, Feb 13, 8:51 PM

	Unknown Object (File)
	Sun, Feb 9, 9:56 AM

	Unknown Object (File)
	Sun, Feb 9, 9:56 AM

	Unknown Object (File)
	Sun, Feb 9, 9:56 AM

	Unknown Object (File)
	Tue, Jan 28, 5:42 AM

	Unknown Object (File)
	Sun, Jan 26, 8:52 PM

	Unknown Object (File)
	Sat, Jan 25, 8:49 AM

	Unknown Object (File)
	Sat, Jan 25, 8:49 AM

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