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

[regression] Missing 0/1/1+ reply indicator with "Show post statistics in timeline" disabled #3588

Closed
1 task done
digitalcircuit opened this issue Apr 28, 2023 · 5 comments · Fixed by #3590
Closed
1 task done
Assignees

Comments

@digitalcircuit
Copy link

digitalcircuit commented Apr 28, 2023

In brief

  • Tusky v22 should show 0/1/1+ reply count indicators with Show post statistics in timeline unchecked

Testing

Steps

  1. Connect Tusky v22.0-beta.1 to an account with various posts on the home timeline
  2. Browse the timeline until you find a toot with replies
  3. Ensure that Show post statistics in timeline is unchecked
  4. Observe the reply icon

Expected

Tusky shows a 0/1/1+ indicator depending on how many replies exist - none/one/more than one.

Screenshot of Mastodon showing the loose reply count indicator

Actual

Tusky won't show any indication of whether a post has replies (not an exact count, just 0/1/1+) until you expand it and wait for the thread to load.

Screenshot of Tusky not showing any indicator next to the reply icon


  • Tusky Version: v22.0-beta.1

  • Android Version: 13 (TQ2A.230405.003.E1)

  • Android Device: Pixel 4 XL

  • Mastodon instance (if applicable): N/A

  • I searched or browsed the repo’s other issues to ensure this is not a duplicate.

@Lakoja
Copy link
Collaborator

Lakoja commented Apr 28, 2023

I've looked at the changes and the number fields in the timeline were only added with them (not changed). At least the ones for boost and favorite.
Also the original issue #2486 says so: "add stats in home timeline"

Taking the wording literally it could be argued that it is correct/consistent at the moment: the opposite of "Show post statistics in timeline" would be "do not show".

Having said that: The reply count was present and it was set to 0/1/1+.
That is now consistent with the other two: Either not showing or showing the actual number.

@nikclayton nikclayton self-assigned this Apr 28, 2023
@nikclayton
Copy link
Contributor

I've looked at the changes and the number fields in the timeline were only added with them (not changed). At least the ones for boost and favorite. Also the original issue #2486 says so: "add stats in home timeline"

Taking the wording literally it could be argued that it is correct/consistent at the moment: the opposite of "Show post statistics in timeline" would be "do not show".

Having said that: The reply count was present and it was set to 0/1/1+. That is now consistent with the other two: Either not showing or showing the actual number.

Yes, but (a) it's still a regression, and (b) the previous functionality for replies was useful, because it told you whether or not the status had one or more replies, so whether or not you should tap through the status to read the replies.

I'm throwing a fix together.

@Lakoja
Copy link
Collaborator

Lakoja commented Apr 28, 2023

b) at least seems a bit twitter-ish: Click here to see the buzz.
In the Fediverse we much more value the intention of the initial post and author.

But I guess it would be consistent with the web ui.

@digitalcircuit
Copy link
Author

@Lakoja

b) at least seems a bit twitter-ish: Click here to see the buzz. In the Fediverse we much more value the intention of the initial post and author.

It might be possible to add some complexity and make the setting a tri-state option, with e.g. "Show nothing", "Show if there are replies", and "Show all stats."

I personally use the 0/1/1+ just to know if it's worthwhile tapping to expand a post, as sometimes my Internet and/or instance can be slow requiring waiting a few seconds. Distinguishing between "1" and "1+" lets me know if someone else replied in the middle of a thread.

(I do yearn for the way Google+ handled comments and how Cohost handles them now, gathering them under a single cohesive post instead of scattered through the timeline, but that's a much larger discussion and change.)

@Lakoja
Copy link
Collaborator

Lakoja commented Apr 28, 2023

@Lakoja
I personally use the 0/1/1+ just to know if it's worthwhile tapping to expand a post, as sometimes my Internet and/or instance can be slow requiring waiting a few seconds. Distinguishing between "1" and "1+" lets me know if someone else replied in the middle of a thread.

As that makes you three and me one that is probably the correct way to go. :-)
I was merely observing anyway.

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 a pull request may close this issue.

3 participants