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

Suggestion: redesign the PostListingList Composable #1465

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rodrigo-fm
Copy link
Contributor

I thought that removing the username, moving the community name to the top and adding the community icon preview would make it cleaner and prettier. Let me know your thoughts on this.

Before

drawing

After

drawing

@MV-GH
Copy link
Collaborator

MV-GH commented Apr 2, 2024

Showing those community avatars should already be an setting that the user can control.

I believe this change is only regarding the list view mode, now my opinion:

List view mode is a compact mode where it only shows what matters for a post, the title. This should be first, second should be the meta data about the post. Your redesign makes it more consistent with the other modes but it reduces the clearity (Not sure how to put this in words but before it was much easier/faster to read the titles)

For some context. Jerboa is heavily influenced by Boost for Reddit in all its UI/UX design aspects. We believe that it has the best UX out of all Reddit clients. And this includes this compact mode which follows BfR.

So I do prefer before but nothing is stopping is us from adding more post view modes but it should be a bigger change than just this.

@rodrigo-fm
Copy link
Contributor Author

I see. I agree that it is more interesting for it to become a separate list item. Can I create this new list item in this pull request?

Honestly I just made it like that because the community icons help me identify faster which community post I'm seeing.

Making it customizable in the settings is a much bigger change.

@MV-GH
Copy link
Collaborator

MV-GH commented Apr 2, 2024

The setting already exists though its show avatar

@dessalines
Copy link
Member

I also lean a tiny bit towards the before, but mainly because I'm partial to boost.

As far as icons: I believe I made that PostListingList just ignore your showAvatar setting in order to make it more compact, but I'd be totally fine with that being added back in.

As far as the other two changes:

  • Removing the creator name: I agree it looks cleaner on mobile, but a lot of people would consider that vital info.
  • Moving the community above the title:
    • This could be solved with a setting that applies to all post views?
    • Or just creating a new post listing view mode.

Its up to you whether you want to:

  • Create a new post view mode, and add your new one. (I like this one the best I think).
  • Or create settings that apply to all post view modes.

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.

None yet

3 participants