-
Notifications
You must be signed in to change notification settings - Fork 66
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 cross-posts #851
Show cross-posts #851
Conversation
Updated with the create cross-post functionality in 8616c54, so I'm taking this out of draft status. qemu-system-x86_64_hhqOOw4Psn.mp4 |
This should be good to go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just had one suggestion! Looks good to me otherwise 😄
I do wonder if we need a setting just to toggle this feature on/off. I feel like most if not all users will keep cross-posts turned on. Regardless, it never hurts to have a setting for it!
lib/post/widgets/post_view.dart
Outdated
child: CommonMarkdownBody( | ||
body: post.body ?? '', | ||
), | ||
), | ||
if (showCrossPosts && sortedCrossPosts.isNotEmpty) | ||
Padding( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be good to separate this into its own widget just for organization purposes (the widget can reside in this file, but just moved out of the PostView widget - to see what I mean by this, you can check out Tagline
on the FeedPage)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea! All set to go now!
Yeah I know, maybe it's overkill. I can just see potential for it being annoying to clutter the post view. 🤷 Regardless it's on by default. Maybe at some point we'll have an advanced settings area for much more granular things that the majority of people won't touch. I can see a lot of the current enhancement requests fitting in this category. 😆 |
Looks good, thanks for the changes! |
Pull Request Description
This PR adds support for displaying, navigating to, and creating cross-posts.
The new UI optionally lists all the communities to which the same link has been posted, allowing you to navigate to any of them. You can also create a new cross-posts, which simply populates the new post UI with the link and title from the existing one.
Issue Being Fixed
Related to #469, but doesn't de-dup. I wonder if we really want de-duping anyway since each version of the post has its own set of unique comments for its community.
Screenshots / Recordings
qemu-system-x86_64_eVUYxYX8FV.mp4
Checklist
semanticLabel
s where applicable for accessibility?