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

Replace video element with static image on Mobile Safari. #441

Closed
wants to merge 1,351 commits into from
Closed

Replace video element with static image on Mobile Safari. #441

wants to merge 1,351 commits into from

Conversation

DFurnes
Copy link
Contributor

@DFurnes DFurnes commented Feb 9, 2015

Changes

For review: @DoSomething/front-end

Discussion

I think this is an appropriate scenario for browser-sniffing, since it's not really a "feature" we're trying to detect but an idiosyncrasy with how a particular browser treats a standard tag. Thoughts?

dsdocs and others added 30 commits November 5, 2014 14:02
Adding height: auto to fix image sizing bug
Remove unused classes & variable.
centered header sizing updates. Fixes 3421
Updates to tile pattern and post gallery button fixes.
Adding margin to h2 tags after ul tags
@sbsmith86
Copy link
Contributor

Looks good! Also, agreed with browser-sniffing for this.

@weerd
Copy link
Contributor

weerd commented Feb 9, 2015

So it's only an idiosyncrasy in a very particular context which applies to us specifically. In any other scenario where video is used on mobile safari, having a click to play makes sense so you don't force a download of the video to an unsuspecting mobile user. It just happens to affect us on the homepage, because our tiles are clickable....

Could there potentially be something we could do with the video controls API with JavaScript? I know you can tap into video controls and affect the visual display of the video controls... maybe that could lead us on the path to a better solution and avoid browser-sniffing? Not sure...

@DFurnes
Copy link
Contributor Author

DFurnes commented Feb 10, 2015

I agree that the non-standard behavior is a usability win for users, but I think it's also a bit unfortunate that Safari ignores the spec for disabling controls on a video element. Ideally they would only ignore the autoplay option.

iOS 8 adds a proprietary CSS pseudo-element that we can target, but that still leaves a visual gotcha for iOS 7 users. I'm all for other solutions, but having trouble finding good ones.

@weerd
Copy link
Contributor

weerd commented Feb 10, 2015

With our powers combined...

@weerd
Copy link
Contributor

weerd commented Feb 10, 2015

we can hopefully find a solution! Oh yeah, and...

@DFurnes DFurnes closed this Apr 1, 2015
@DFurnes DFurnes deleted the tile-video-mobile branch April 1, 2015 21:13
@DFurnes DFurnes restored the tile-video-mobile branch April 1, 2015 21:13
@DFurnes DFurnes reopened this Apr 1, 2015
@DFurnes DFurnes force-pushed the dev branch 2 times, most recently from 2109e12 to 56344f1 Compare August 14, 2015 15:55
@DFurnes DFurnes closed this Aug 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Tile with a video should not show "play" button on mobile
5 participants