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

Show toot stat inline #3413

Merged
merged 17 commits into from
Mar 18, 2023
Merged

Show toot stat inline #3413

merged 17 commits into from
Mar 18, 2023

Conversation

zikasak
Copy link
Contributor

@zikasak zikasak commented Mar 3, 2023

Resolves #2486

@connyduck
Copy link
Collaborator

  • Needs some kind of clever number formatting to make sure the labels don't get too long, e.g. instead of 123456 display 123k.
  • Please add a setting to turn this on and off, default to off

@zikasak
Copy link
Contributor Author

zikasak commented Mar 8, 2023

I've added formatting and a setting

@zikasak
Copy link
Contributor Author

zikasak commented Mar 10, 2023

Screenshot_20230310_144057_Tusky Test copy

Here is an example. The reply counter was mocked to show how it looks on big numbers

@Lakoja
Copy link
Collaborator

Lakoja commented Mar 10, 2023

Could probably do with a bit more margin to the images.

Otherwise it looks fine.

@zikasak
Copy link
Contributor Author

zikasak commented Mar 10, 2023

Screenshot_20230310_151116_Tusky Test
Is it ok with such a margin?

@Lakoja
Copy link
Collaborator

Lakoja commented Mar 10, 2023

That looks good.

@Lakoja
Copy link
Collaborator

Lakoja commented Mar 10, 2023

(Old PR that can be closed then: #1369)

@zikasak zikasak marked this pull request as ready for review March 11, 2023 06:41
Copy link
Collaborator

@connyduck connyduck left a comment

Choose a reason for hiding this comment

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

I like how it works now, code needs a bit of improvement then I'll merge it


private void setFavouritedCount(int favouritedCount) {
if (favouritedCountLabel != null) {
favouritedCountLabel.setText(NumberUtilsKt.shortNumber(favouritedCount));
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since it does not affect conversations or detailed statuses, all these changes should be in StatusViewHolder. No more null check necessary then.

@connyduck connyduck merged commit 75e7b9f into tuskyapp:develop Mar 18, 2023
@nikclayton
Copy link
Contributor

@zikasak -- I'm writing the changelog for v22, which includes this PR, and (optionally) a link to your Mastodon profile. If you'd like that to be added please can you send it to me. Thanks.

@scollovati
Copy link

May I suggest to remove the 0 (zero) indicator? Adding the number only when different from zero increases readability.

@Tak Tak mentioned this pull request Sep 4, 2023
1 task
Tak pushed a commit that referenced this pull request Sep 26, 2023
The idea here is: Everytime we get hold of a new version of a post, we
update everything about that post everywhere.
This makes the distincion between different event types unnecessary, as
everythng is just a `StatusChangedEvent`.
The main benefit is that posts should be up-to-date more often, which is
important considering there is now editing and #3413
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.

[feature request] add stats in home timeline
5 participants