Page MenuHomePhabricator
Differential D21508

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
F13167320: D21508.diff
Tue, May 7, 7:04 AM
Unknown Object (File)
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
View All 40 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
Branch
return1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25006
Build 34499: Run Core Tests
Build 34498: arc lint + arc unit

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

PathSize
src/
infrastructure/
markup/
rule/
16 lines

Diff 51193

View Options

src/infrastructure/markup/rule/PhabricatorKeyboardRemarkupRule.php

Loading...
Phabricator · Built by Phacility