Show Favs and Repeats in the statuses #202

Closed
opened 2018-12-07 12:14:44 +00:00 by lambadalambda · 30 comments

It would be nice to be able to view favs and RTs in the statuses, not just the number. Maybe have like 5 small round avatars for each max, with an option to expand to see all people who fav'd it. We have this functionality on the backend already, but it's not used outside of api clients.

It would be nice to be able to view favs and RTs in the statuses, not just the number. Maybe have like 5 small round avatars for each max, with an option to expand to see all people who fav'd it. We have this functionality on the backend already, but it's not used outside of api clients.
Member

I was just thinking that today!

I was just thinking that today!
Member

How about we change layout of post a bit when it's selected? That would also allow to get rid of different background color for "selected" post.

How about we change layout of post a bit when it's selected? That would also allow to get rid of different background color for "selected" post.
Author
Owner

makes sense to me, that's how qvitter did it as well.

makes sense to me, that's how qvitter did it as well.
Author
Owner

Easiest first step would be to just show it for the selected post in an expanded conversation

Easiest first step would be to just show it for the selected post in an expanded conversation
Member

This feature can be split into 2 main sub-features :

  • The feature we are talking about here
  • A modal view which appears when the user clicks on where we display the list of all users favs/RT
This feature can be split into 2 main sub-features : - The feature we are talking about here - A modal view which appears when the user clicks on ➕ where we display the list of all users favs/RT
Member

yeah, i was thinking about making it a dropdown/popup for extra users.

'cause modals are evil

yeah, i was thinking about making it a dropdown/popup for extra users. 'cause modals are evil
Member

Fair enough :)

Fair enough :)
Member

@lambadalambda QvitterAPI doesn't have the necessary APIs for it, and there's seem to be no relevant data in /api/statusnet/conversation/######.json

@lambadalambda QvitterAPI doesn't have the necessary APIs for it, and there's seem to be no relevant data in `/api/statusnet/conversation/######.json`
Owner

If someone can open an issue in the backend with details on what API is required and we can put someone to work on this

If someone can open an issue in the backend with details on what API is required and we can put someone to work on this
Author
Owner

You can use the mastodon api for this, we already have the necessary endpoints there: https://docs.joinmastodon.org/api/rest/statuses/#get-api-v1-statuses-id-reblogged-by

You can use the mastodon api for this, we already have the necessary endpoints there: https://docs.joinmastodon.org/api/rest/statuses/#get-api-v1-statuses-id-reblogged-by

Screen_Shot_2019-03-29_at_12.04.38_PM

guys, this is ui implementation from the ui and i put the component in jsx to show how it looks.
At the moment, it slices 10 users from provided users and display in stacked avatar list.
We are supposed to show the number of replies and favs so I'd like to get an idea where to display this avatar and to display rest of the users

![Screen_Shot_2019-03-29_at_12.04.38_PM](/attachments/d3d79001-112f-447c-9966-edf169b924d3) guys, this is ui implementation from the ui and i put the component in jsx to show how it looks. At the moment, it slices 10 users from provided users and display in stacked avatar list. We are supposed to show the number of replies and favs so I'd like to get an idea where to display this avatar and to display rest of the users
Owner

How should we display this? Should we do hover over the number for each type (reply, boost, fav)? Should we show avatars for all combined?

Any other ideas?

How should we display this? Should we do hover over the number for each type (reply, boost, fav)? Should we show avatars for all combined? Any other ideas?
Member

Is this stacked avatar list also shown in the timelines? I am not ui designer but it seems like a lot of wasted space for something most of us wouldn't even want to take a look at

Is this stacked avatar list also shown in the timelines? I am not ui designer but it seems like a lot of wasted space for something most of us wouldn't even want to take a look at
Member

I agree with @rinpatch. They can be smaller and more collapsed.

I agree with @rinpatch. They can be smaller and more collapsed.
Member

Sorry to say but I never fancy the existing "+" icon. I think it's a misleading icon. We should either correct that to

  1. A global "..." icon that indicates "more". With that you can easily add extra functions within the status in the future.
    OR
  2. Remove it

In that case you can show the expandable content

  1. While people click on the "..." icon
    OR
  2. Click on the body of the posted status

Finally, we need to decide, if we show avatar from favs or repeats.

Sorry to say but I never fancy the existing "+" icon. I think it's a misleading icon. We should either correct that to 1. A global "..." icon that indicates "more". With that you can easily add extra functions within the status in the future. OR 2. Remove it In that case you can show the expandable content 1. While people click on the "..." icon OR 2. Click on the body of the posted status Finally, we need to decide, if we show avatar from favs or repeats.
Author
Owner

This isn't in the timeline view, it has to be fetched for every status so it only makes sense in a special kind of interaction, like an expanded status.

This isn't in the timeline view, it has to be fetched for every status so it only makes sense in a special kind of interaction, like an expanded status.
Author
Owner

maybe even hovering over the number, then expanding it.

maybe even hovering over the number, then expanding it.
Owner

Is this stacked avatar list also shown in the timelines? I am not ui designer but it seems like a lot of wasted space for something most of us wouldn't even want to take a look at

I plan this to be for a single status or the expanded thread right now. Later we can consider some kind of "hover over the numbers" feature in timelines but if we do that poorly it will make everyone furious with the popup obscuring important screen real estate

Finally, we need to decide, if we show avatar from favs or repeats.

We are using the Conversations API endpoint for this which includes users of all related activities: favs, boosts, replies.

> Is this stacked avatar list also shown in the timelines? I am not ui designer but it seems like a lot of wasted space for something most of us wouldn't even want to take a look at I plan this to be for a single status or the expanded thread right now. Later we can consider some kind of "hover over the numbers" feature in timelines but if we do that poorly it will make everyone furious with the popup obscuring important screen real estate > Finally, we need to decide, if we show avatar from favs or repeats. We are using the Conversations API endpoint for this which includes users of *all* related activities: favs, boosts, replies.
Owner

I think the text in the image is redundant and should be removed, so just ignore that for now.

I think the text in the image is redundant and should be removed, so just ignore that for now.
Member

Something like this.
image

also probably should truncate list to show X avatars at max, with showing all in modal or in-line on click with maybe "show more" link?

Something like this. ![image](/attachments/e01c50de-f483-493c-aa16-3728ef8aa3f1) also probably should truncate list to show X avatars at max, with showing all in modal or in-line on click with maybe "show more" link?
Member

not saying that avatars should look like this, they should not have ugly border and should have notification avatar roundness

not saying that avatars should look like this, they should not have ugly border and should have notification avatar roundness
Member

I think it gives likes and boosts much more importance than they actually deserve, especially if it is under every single post in the thread, why not show them on hover like here? (absolutely proprietary screenshot yes)

2019-03-29-201621

I think it gives likes and boosts much more importance than they actually deserve, especially if it is under every single post in the thread, why not show them on hover like here? (absolutely proprietary screenshot yes) ![2019-03-29-201621](/attachments/5c44ba23-59d7-45cc-a43f-d786991850e8)
Member

So how would the hover react on mobile version?

So how would the hover react on mobile version?
Member

Like reply previews? Just do the same thing but on click

Like reply previews? Just do the same thing but on click

I agree with @rinpatch (display avatars on certain event)
To displaying avatars on initial loading doesn't make sense since it needs sequential api call to favorited_by and reblogged_by to get avatar information which will cause performance issue.
I'd propose to show avatars in popup on hover event (click on mobile)

I agree with @rinpatch (display avatars on certain event) To displaying avatars on initial loading doesn't make sense since it needs sequential api call to favorited_by and reblogged_by to get avatar information which will cause performance issue. I'd propose to show avatars in popup on hover event (click on mobile)
Owner

This data is useless if you have to hover or expand something to see it. I don't see any performance issues in the iOS Toot app when it displays the avatars in expanded statuses/threads

This data is useless if you have to hover or expand something to see it. I don't see any performance issues in the iOS Toot app when it displays the avatars in expanded statuses/threads
Owner

I do see that the avatars take a while to load or fail in poor network conditions.C4227A64-E351-446F-8202-81A2E7BF25EEC15E814D-35E8-4F0E-8489-547438825CEF

I do see that the avatars take a while to load or fail in poor network conditions.![C4227A64-E351-446F-8202-81A2E7BF25EE](/attachments/808b54f6-c5db-4011-9ab4-4daa65e58294)![C15E814D-35E8-4F0E-8489-547438825CEF](/attachments/c2416967-2369-4326-9220-dbcba168f9f9)
Member

This data is useless if you have to hover or expand something to see it

I really don't get why is it so useful it should waste additional screen space, in half a year of being on fediverse I wanted to know up who liked the post like, 4 times. And every fedi app I used (MastoFE, Fedilab, Twidere, Tusky) hides it behind a click/hover

> This data is useless if you have to hover or expand something to see it I really don't get why is it so useful it should waste additional screen space, in half a year of being on fediverse I wanted to know up who liked the post like, 4 times. And every fedi app I used (MastoFE, Fedilab, Twidere, Tusky) hides it behind a click/hover
Owner

Users love this, makes the experience richer and more modern looking. If we catered to what only techy users want we'd probably be using plaintext interfaces for maximum use of screen real estate...

Users love this, makes the experience richer and more modern looking. If we catered to what only techy users want we'd probably be using plaintext interfaces for maximum use of screen real estate...
Member

Not true. Completely not true. I'm a techy user but I like it. I'm just picky.

Not true. Completely not true. I'm a techy user but I like it. I'm just picky.
Sign in to join this conversation.
No milestone
No project
No assignees
8 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
pleroma/pleroma-fe#202
No description provided.