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

add modal message when editing toot #23936

Conversation

PauloVilarinho
Copy link
Contributor

closes #22851

This PR adds a modal when there is text on the compose form and the user clicks on the edit button on another toot.
mastodon_edit
I was not able to find where to add tests for this one. If anyone can help me with this problem I would be very grateful.

Comment on lines 165 to 166
"confirmations.edit.confirm": "Editar",
"confirmations.edit.message": "Editar agora sobrescreverá o toot que você está compondo. Deseja continuar?",
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good overall, but changing translation files directly runs a risk of creating merge conflicts with the automated Crowdin PRs. It's safer to change only en.json in PRs and then change the translations in Crowdin.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok I'm fixing this

@PauloVilarinho PauloVilarinho force-pushed the add-modal-messages-when-editing-toot branch from e65e289 to e28303f Compare March 6, 2023 15:58
@ClearlyClaire ClearlyClaire merged commit 8770473 into mastodon:main Mar 6, 2023
12 checks passed
ClearlyClaire pushed a commit that referenced this pull request Mar 14, 2023
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
rutvijmehta-harness pushed a commit to rutvijmehta-harness/mastodon that referenced this pull request Mar 19, 2023
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
rutvijmehta-harness added a commit to rutvijmehta-harness/mastodon that referenced this pull request Mar 19, 2023
Co-authored-by: PauloVilarinho <33267902+PauloVilarinho@users.noreply.github.com>
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
arachnist pushed a commit to arachnist/mastodon that referenced this pull request Apr 4, 2023
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
Roboron3042 pushed a commit to Roboron3042/mastodon that referenced this pull request Apr 16, 2023
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
skerit pushed a commit to 11ways/mastodon that referenced this pull request Jul 7, 2023
Co-authored-by: PauloVilarinho <paulotarsobranco@hotmail.com>
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.

"Edit post" should warn before overwriting text in the post creation box
2 participants