-
-
Notifications
You must be signed in to change notification settings - Fork 540
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cmd-S not tied to Save on Navigation CP #1794
Comments
Confirmed. You must add an item to recreate the issue - simply rearranging the nav doesn't cause issue. |
Can't quite find the fix on this one, but it's only happening when you add a URL, not a link. |
Hu guys, I looked into this one. The problem is that the included PageEditor component is stopping the events necessary for the keyboard shortcuts to work. The code in PageEditor could be improved to avoid that. I'll create a PR with the fix. |
Bug Description
If you're on the Navigation screen and you have changes and hit cmd-s (like you can do on entries), you don't save the data, you get the browser save.
How to Reproduce
Environment
Statamic version: 3.0.0-beta-27
PHP version: 7.4
Install method (choose one):
statamic/statamic
The text was updated successfully, but these errors were encountered: