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

avoid saving editor document title without changes #452

Merged
merged 6 commits into from Apr 20, 2023

Conversation

lagartoverde
Copy link
Contributor

Solves https://binnenland.atlassian.net/browse/GN-4208, I couldn't reproduce the issue on local while I can easily do it on dev and qa, so it must have been fixed by other PR in the meantime.
I solved the issue where you could save a title even if it was unchanged

Copy link
Member

@abeforgit abeforgit left a comment

Choose a reason for hiding this comment

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

the ticket also asks for an indication that it's been saved. So I'd suggest playing the same saving animation as for the document (but only IF it also saves the document)

@lagartoverde
Copy link
Contributor Author

I added a "Saved" AuPill next to the title, that hides itself after 30s to show the user the title got saved. Double check the dutch translation, I don't know if "Gered" is correct here, "Opgeslagen" was also suggested by Deepl

@lagartoverde lagartoverde merged commit 77baa56 into master Apr 20, 2023
1 check passed
@lagartoverde lagartoverde deleted the bugfix/avoid-saving-title-without-change branch April 20, 2023 10:03
@lagartoverde lagartoverde added the enhancement New feature or request label Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants