-
Notifications
You must be signed in to change notification settings - Fork 913
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
(#6716) Update video placeholders #6717
(#6716) Update video placeholders #6717
Conversation
- updated video placeholder images for videos - compressed fbvideochat images since I was in there
I can't remember why this video is different to the rest in the way it uses a custom placeholder overlay, and not a native one. There are probably historical reasons why it's like this, but I'm not sure those reasons still exist. Could removing the custom placeholder and making the top video uniform with the other videos be a simple solution? (less code, too). Edit: could probably remove this test, too. |
OMG yes. I assumed it was for a good reason but if there isn't one I'll update it :) |
2f9a72b
to
e13b567
Compare
Updated to remove custom placeholder html, script, and test for speed video. |
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.
Did you mean to delete fbvideochat-screenshot.jpg
? This still seems to be in use.
The video test also needs removing https://github.com/mozilla/bedrock/blob/master/tests/functional/firefox/test_home.py#L20
I don't think I deleted |
Updated to remove mute and test. |
Description
Issue / Bugzilla link
#6716
Testing