Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

hide link previews when a status is hidden behind a content warning (fixes issue #95) #114

Merged
merged 2 commits into from
Nov 10, 2022

Conversation

dvorakroth
Copy link
Contributor

@dvorakroth dvorakroth commented Nov 8, 2022

Summary

hi! this seemed like an extremely easy fix, so i thought i'd give it a shot 😅 i hope i didn't miss anything lol!

screenshots of what the fix looks like:

screenshot of a post by @dog, with a content warning reading 'woof woof!' and a 'show more' button

screenshot of the same post as the previous image, but the button now says 'show less', and the post's body is now shown: both the text reading 'arwoo!' and the link preview card to the wikipedia article for Dog

fixes issue #95

Other Information

@prplecake
Copy link
Contributor

As mentioned in #58, this doesn't fix the issue.

@dvorakroth
Copy link
Contributor Author

that's.... really weird. i can indeed confirm that the latest app store version still has the problem -- but when i take the latest code and run it on the iphone simulator on my computer, the fix works perfectly!

i can't seem to debug the latest code on an actual device, though (getting some errors that i'm not sure how to fix; would love some help if anyone has the time and know-how!) so i can't rule out that there may be some weird bug-within-a-bug that only happens on real devices?

@prplecake
Copy link
Contributor

I suppose it's possible the app store build isn't actually up to date... but that's pure speculation. I've never personally shipped anything to the app store.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants