Skip to content

Conversation

@JuliaKirschenheuter
Copy link
Contributor

@JuliaKirschenheuter JuliaKirschenheuter commented Jan 19, 2023

After:

image

Summary

Add success message to personal settings button

Checklist

@JuliaKirschenheuter
Copy link
Contributor Author

/backport to stable25

}
await navigator.clipboard.writeText(this.cloudId)
this.isCopied = true
showSuccess(t('federatedfilesharing', 'Copied!'))
Copy link
Member

Choose a reason for hiding this comment

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

is the text "Copied!" enough or should it be a longer message with context ?

Copy link
Member

Choose a reason for hiding this comment

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

We use it like this everywhere

@ChristophWurst ChristophWurst changed the title Add success message to personal settings button Add success message to personal settings clipboard button Jan 23, 2023
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 23, 2023
@nickvergessen nickvergessen merged commit 1515239 into master Jan 23, 2023
@nickvergessen nickvergessen deleted the fix/add-success-message-to-personal-settings-button branch January 23, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants