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 ability to block account from post dropdown menu #6091

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rshigg
Copy link

@rshigg rshigg commented Nov 3, 2024

Details

While #3708 already exists, it hasn't been updated sine April and I personally don't think including the mute option is necessary. If it is decided that a mute button is needed/wanted, I can update this PR or I could open a separate PR.

  • I matched the blocking flow from the profile header but made it one-directional.
  • I grouped the block option with the report option because they are both moderation actions.

Note: I don't consider this PR complete since I assume we will want some sort of visual feedback that a user was blocked while in a feed. My initial thinking was to hide the post after blocking but it felt a little janky. I would appreciate some feedback.

Demo

(Sorry Abby and Jerry, it's not personal 😅 )

From a feed From a profile page
block_from_feed.webm
block_from_profile.webm
Screen.Recording.2024-11-03.at.12.34.53.PM.mov
Screen.Recording.2024-11-03.at.12.35.13.PM.mov

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.

1 participant