QOL Remove unnecessary clickable areas from PostListingList #710
Conversation
The PostListingList is too small to have multiple clickable areas, leading many users to accidentally click into the community or the users profile from the front page instead of opening the post like they intended.
| showDefaultIcon = false, | ||
| ) | ||
| DotSpacer(0.dp) | ||
| } | ||
| PersonProfileLink( | ||
| person = postView.creator, | ||
| isModerator = isModerator, | ||
| onClick = onPersonClick, | ||
| onClick = {}, |
There was a problem hiding this comment.
I don't think this is a good idea. Now users won't be able to go to a person's profile, or a community, from the post listing, at all.
@twizmwazin thoughts?
There was a problem hiding this comment.
I don't think people need or want to go to a users profile from the front page. If they want to, they can open the post and click through from there.
Take a look at the linked discussion thread, over 100 upvotes and almost 20 people commenting that this is also annoying them.
There was a problem hiding this comment.
There was na issue opened about this, I'll link it here if I can find it. Users (including myself) accidentally fat-finger the small touch targets frequently and it is a bit of an annoyance. It is simple enough to open the post and then click the appropriate link from in there. An option wouldn't be a bad idea, which is what I think Boost does.
There was a problem hiding this comment.
mmmk, I spose its fine, especially since this just affects the list version.
|
I think an increase in the Heading font size would accomplish this without changing anything else?
|
|
@ZJouba You're right, but there are many users who prefer a more compact layout, myself included, who would prefer this approach. Settings in the future could be a nice way to accommodate both preferences. |
| showDefaultIcon = false, | ||
| ) | ||
| DotSpacer(0.dp) | ||
| } | ||
| PersonProfileLink( | ||
| person = postView.creator, | ||
| isModerator = isModerator, | ||
| onClick = onPersonClick, | ||
| onClick = {}, |
There was a problem hiding this comment.
mmmk, I spose its fine, especially since this just affects the list version.
|
With these changes, does clicking over a username or community name go to the underlying post instead, or does it just ignore the click and do nothing? I think the former would be a better experience for users. |
Clicking a username opens the post |
The PostListingList is too small to have multiple clickable areas, leading many users to accidentally click into the community or the users profile from the front page instead of opening the post like they intended.
https://lemmy.ml/post/1303517
Fixes #488