Skip to content
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

fix(ui): blocks browser save dialog from opening when hotkey used with no changes #6365

Merged
merged 2 commits into from
May 21, 2024

Conversation

JessChowdhury
Copy link
Member

@JessChowdhury JessChowdhury commented May 15, 2024

Description

Related issue #214 (3.0).

Using the cmd+s hotkey in the Edit Document view was opening the browser save dialogue when no changes had been made.

This change triggers a toast info banner with No changes to save and adds translated string for other languages.

PR for 3.0 is here.

  • I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

  • Chore (non-breaking change which does not add functionality)

Checklist:

  • Existing test suite passes locally with my changes

@JessChowdhury JessChowdhury changed the title chore: triggers toast banner if cmd+s used when no document changes fix(ui): blocks browser save dialog from opening when hotkey used with no changes May 21, 2024
@JessChowdhury JessChowdhury merged commit 8f03cd7 into main May 21, 2024
33 checks passed
@JessChowdhury JessChowdhury deleted the chore/save-hotkey branch May 21, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant