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

Don't remove originals of boosted toots from timeline #5479

Merged
merged 2 commits into from
Nov 24, 2017

Conversation

duxovni
Copy link
Contributor

@duxovni duxovni commented Oct 20, 2017

I believe this fixes #5097 . I'd like to hear from @Gargron what the original rationale was for including this line, though.

@Gargron
Copy link
Member

Gargron commented Oct 31, 2017

Do people want to see duplicate original+reblog in their home?

@nullkal
Copy link
Contributor

nullkal commented Oct 31, 2017

Some people toots after boost when they want to talk something related to the boosted toot.

So, I want to see the same toot whenever some followee boosted.

@nolanlawson
Copy link
Contributor

I find it annoying when the toot disappears after I boost it, yeah. I always have to remember to favorite it before boosting, and as @nullkal says you can't reply after boosting without finding the toot all over again. I think maybe in this one case duplicates can be tolerated.

@nolanlawson
Copy link
Contributor

OTOH this PR appears to duplicate all reblogs, not just the one reblogged by the user themselves?

@duxovni
Copy link
Contributor Author

duxovni commented Oct 31, 2017

I think that this PR is a good idea even if we're not dealing with toots the user boosted themselves. If I'm about to click the "favorite" button on a toot, and someone I'm following boosts that toot a second before I click, the toot will disappear from under my mouse and I'll click on some other random toot. If we don't want duplicates, then rather than removing the original we should just not show the boost.

@duxovni
Copy link
Contributor Author

duxovni commented Oct 31, 2017

But yup, as @nullkal points out, not showing the boost would make "LB" posts more confusing, so I think it's fine to just have duplicates.

@duxovni
Copy link
Contributor Author

duxovni commented Oct 31, 2017

Also, I believe that we still won't have duplicates in close proximity, because if a toot is already near the top of your timeline then boosts don't seem to get added to the timeline at all.

@duxovni
Copy link
Contributor Author

duxovni commented Nov 6, 2017

Just checking in, are there any objections to this PR that haven't been addressed?

@charlag
Copy link

charlag commented Nov 14, 2017

I agree, current behavior is counter-intuitive and I would rather see duplicates or see the original post.

Copy link
Contributor

@akihikodaki akihikodaki left a comment

Choose a reason for hiding this comment

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

I'm for this change. Disappearing toots are too annoying.

@akihikodaki
Copy link
Contributor

Do people want to see duplicate original+reblog in their home?

Reblogs made in the same session do not have to be inserted anyway.

@Gargron Gargron merged commit 2b51b40 into mastodon:master Nov 24, 2017
cobodo pushed a commit to cobodo/mastodon that referenced this pull request Dec 6, 2017
* Don't remove originals of boosted toots from timeline

* Remove unused argument to updateTimeline
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.

Boosted toots jump to top of timeline
6 participants