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

Change confirm prompt for relationships management #19411

Merged
merged 3 commits into from Jan 14, 2023

Conversation

tribela
Copy link
Contributor

@tribela tribela commented Oct 22, 2022

On relationships manager, Just asking same "Are you sure?" for destructive action is not good.
Especially, for mobile interface, User cannot know if they touched different action

It maybe better if "Are you sure to ~~(action) for X users?", But it is fine with confirming with action like "Are you sure to follow selected followers?"

@ineffyble ineffyble added ui Front-end, design moderation Administration and moderation tooling labels Nov 14, 2022
Copy link
Contributor

@TobyWilkes TobyWilkes left a comment

Choose a reason for hiding this comment

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

PR is currently marked as approved, but the en yml has syntax issues

config/locales/en.yml Outdated Show resolved Hide resolved
config/locales/en.yml Outdated Show resolved Hide resolved
config/locales/en.yml Outdated Show resolved Hide resolved
@ClearlyClaire
Copy link
Contributor

@tribela can you rebase it on latest main?

Co-authored-by: TobyWilkes <tobylwilkes@gmail.com>
@ClearlyClaire ClearlyClaire merged commit d66dfc7 into mastodon:main Jan 14, 2023
@tribela tribela deleted the relationships-messages branch January 17, 2023 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
moderation Administration and moderation tooling ui Front-end, design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants