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

By default, hide replies to people you don't follow in the Following feed #3124

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Mar 6, 2024

Complement to bluesky-social/atproto#2279.

It has been a pretty common complaint that the Following feed becomes unusable when someone you follow goes on a reply spree because we're showing every single reply by default. This changes so that by default (if you haven't expressed a preference), we're going to show a reply if you follow both people in the conversation.

People who have expressed a particular preference will keep it.

Test Plan

Create a new user or take an existing one who hasn’t changed this setting. Observe that their default preferences have the "Reply Filters" toggle on whereas previously they wouldn’t.

Screenshot 2024-03-06 at 14 56 45

See the behavior difference below.

Also verified that if you explicitly chose a setting, your preference remains respected.

Before

A user who hasn't changed the setting would see replies to people they don't follow in their Following feed.

following_before.mov

After

A user who hasn't changed their setting would see only reply chains to people they follow in their Following feed.

following_after.mov

@gaearon gaearon changed the title Show replies from followed by default By default, hide replies to people you don't follow in the Following feed Mar 6, 2024
@gaearon gaearon requested a review from pfrazee March 6, 2024 17:22
@gaearon gaearon merged commit 5b8d116 into main Mar 6, 2024
4 checks passed
@gaearon gaearon deleted the default-reply-followed branch March 6, 2024 17:28
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.

3 participants