-
Notifications
You must be signed in to change notification settings - Fork 97
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
Pages with Videos are Broken in Mobile View #55
Comments
@jusuchin85 Thanks for reporting this issue! I will investigate right now and see if I can get a quick patch fix in. |
Great! Thanks, @kelyvin ! |
Hi @jusuchin85, I think I fixed the issue. Can you try pulling the latest release here (https://github.com/kelyvin/caffeine-theme/releases/tag/v3.0.4) and giving it a try? Youtube seems to be doing some code injection on video embedding, which dynamically tries to style the page. This seems to have caused issues, so I overrode their style changes with my own. Let me know if this fixes your problem! It seems to be fixed with my post (https://www.caffeinecoding.com/google-io-2016-in-review/). |
Thanks, man! I'll give it a try again when I reach home. On the train back
now.
|
Thanks @kelyvin ! The fix works! |
Hey @kelyvin ,
Notice this on mobile view when there's a Youtube video attached:
When viewing the same page on a desktop browser, you can see the heading text after the same Youtube embed:
The text was updated successfully, but these errors were encountered: