-
Notifications
You must be signed in to change notification settings - Fork 68
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
Added an option to hide thumbnails in feed #1302
Added an option to hide thumbnails in feed #1302
Conversation
Thanks for doing this @iparks99! I just have a discussion point for this. It might be worth it to extend this to both compact/card views. For compact mode, you already have it implemented which is great. I'm thinking for the card view, we can replace the image with just a link (see the image for reference). The link can then distinguish the type of media based on the leading icon (link, image, video) If we do go forward with this, we can move the setting to the general setting area in appearance (Appearance > Posts > Feed Settings). @micahmo do you have any thoughts on this idea? |
Sorry I hadn't looked too closely at this yet since it was a draft. I do like your idea of making this a general setting for all post types and using the link placeholder to indicate the media type in card view. As long as the link placeholder can act as the media preview in the sense that tapping it will show the media (if applicable). And we definitely need something similar to show the media type in condensed view as well. |
f4e4158
to
cb96e26
Compare
Thanks for the feedback @hjiangsu and @micahmo! I've done the following based on your feedback:
hide-thumbnails-demo-2024-04-19_16.18.45.mp4 |
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.
I left a couple small comments, but otherwise LGTM. Nice work!!
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 some minor code documentation changes. Otherwise, LGTM!
Pull Request Description
Added a toggle option to Post Compact View Settings to hide thumbnails entirely from the feed. This does not affect images in post bodies.
Potential To Do:
An option to hide images in post bodiesIssue Being Fixed
Not loading images in the feed would be advantageous to anyone with limited data, or just preferring a text-only feed.
Issue Number: #352
Screenshots / Recordings
Checklist
semanticLabel
s where applicable for accessibility?