Skip to content

Enable dev keyboard shortcuts on Mac Catalyst#27479

Closed
charpeni wants to merge 1 commit into
react:masterfrom
charpeni:devmenu-on-catalyst
Closed

Enable dev keyboard shortcuts on Mac Catalyst#27479
charpeni wants to merge 1 commit into
react:masterfrom
charpeni:devmenu-on-catalyst

Conversation

@charpeni

Copy link
Copy Markdown
Contributor

Summary

This enables the dev menu to bo opened from keyboard shortcuts in dev from a Mac Catalyst app.

cc @TheSavior @andymatuschak @radex

Changelog

[iOS] [Fixed] - Enable dev keyboard shortcuts on Mac Catalyst

Test Plan

It depends on #27469 (to have working WebSocket in debug).

image

@charpeni charpeni requested a review from shergin as a code owner December 11, 2019 14:26
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. labels Dec 11, 2019

@andymatuschak andymatuschak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I suspected it was something like this, but I failed to track it down in #27469. Thank you! These changes do indeed appear to be what's required.

@hramos hramos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. While I think it'd be fine to merge now, I'll wait for #27469 to be reviewed and merged before landing these changes.

@ecreeth

ecreeth commented Jan 24, 2020

Copy link
Copy Markdown
Contributor

Ping! #27469 was merged!

@facebook-github-bot facebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shergin is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @charpeni in 56dfc86.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Jan 27, 2020
osdnk pushed a commit to osdnk/react-native that referenced this pull request Mar 9, 2020
Summary:
This enables the dev menu to bo opened from keyboard shortcuts in dev from a Mac Catalyst app.

cc TheSavior andymatuschak radex

## Changelog

[iOS] [Fixed] - Enable dev keyboard shortcuts on Mac Catalyst
Pull Request resolved: react#27479

Test Plan:
It depends on react#27469 (to have working WebSocket in debug).

![image](https://user-images.githubusercontent.com/7189823/70629346-d3a68880-1bf7-11ea-8949-7553157a2f9c.png)

Differential Revision: D19576528

Pulled By: shergin

fbshipit-source-id: 32b4f8424fb7d270640af4bc50dba24f488bef4f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API: Keyboard Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Merged This PR has been merged. Platform: iOS iOS applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants