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

Basic implementation of TikTok embeds #782

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

JoeGermuska
Copy link
Member

Addresses #781

The embeds basically work, although there is a painfully long lag in loading, and there doesn't seem to be any way to control the height of the embed, which means it overflows the available space on a typical timeline.

Will let this sit for a little to see if we (or anyone watching!) can figure those out, but I don't think they're blockers to integrating it.

…ntrol the height, so the embed overflows, which is annoying.
@JoeGermuska
Copy link
Member Author

After fooling around in the browser inspector, it seems like it's things in the embed which force the height to 740px. Since we can't control the internals, it seems that people will simply have to tolerate that the tiktok embed will, in most cases, overflow the "StorySlider" and the bottom will be covered by the navigation component.

@JoeGermuska JoeGermuska marked this pull request as ready for review July 28, 2022 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant