-
Notifications
You must be signed in to change notification settings - Fork 613
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
Crashes on iPad #134
Comments
Hi Rachel, I have two answers for you. First, let's try to solve your problem today. https://gist.github.com/imakewebthings/1342565 The problem you're having has been a part of deck.js from day one. The snippets in the gist above is a hacked together extension to work around the problem. If you include it in your deck, I believe it will help. Here's how it basically works:
So hopefully you can apply this to your project and things smooth out. Keep me updated on whether it works or not. The second answer I have is that the first answer is a shitty answer. This is a consideration that needs to be moved into the core or core set of default extensions. The git log of this project hints at exactly how little time I've had to work on deck.js recently. But I am trying earnestly to carve out a large chunk of time in the near future to both work towards clearing the issue backlog and to implement major changes. This issue needs to be a part of that work. So even if the above solution works for your deck, I'm going to keep this issue open and marked until it's resolved properly. Thanks, and good luck. |
Wow, thanks for getting back to me so comprehensively! My only question is: how do I initiate this alongside ye olden: $.deck('.slide'); I'm not familiar with doing much extending! |
@rachelnabors You could put the JS portion inside its own JS file and load it with a |
So, big problem here: when I skip directly to a page with a hashtag like so http://rachelnabors.com/css-amv-talk/animation-power-techniques.html#slide-29 the page renders blank... |
@rachelnabors Am I right that the page you linked to doesn't include the horizon stuff? |
It does not, sorry! On a plane. Here, rachelnabors.com/css-animation-master-class/index.html#slide-222 should do it. Rachel Nabors Coming to speak at a city near you! RachelNabors.com (http://rachelnabors.com/) | @rachelnabors (http://twitter.com/rachelnabors) | my old comics (http://rachelthegreat.com/) On Thursday, October 3, 2013 at 7:43 PM, Caleb Troughton wrote:
|
@rachelnabors Underscoring the hackiness of this stopgap further... could you please try taking the horizon chunk of script and moving it to above the script that includes deck.hash? Leaving the $.deck('.slide') init where it is... Thanks. |
That seems to have worked! Thanks! Rachel Nabors Coming to speak at a city near you! RachelNabors.com (http://rachelnabors.com/) | @rachelnabors (http://twitter.com/rachelnabors) | my old comics (http://rachelthegreat.com/) On Thursday, October 3, 2013 at 9:55 PM, Caleb Troughton wrote:
|
I wanted to document this behavior. I've checked other, shorter deck.js slides that don't have this problem. But on the iPad, my large slide decks like https://github.com/rachelnabors/animation-workshop will crash iOS Safari. I think this has to do with the number of iframes. I'm looking for a solution, including investigating the lazyload extension (but I don't like how it interrupts deck.js's elegant pagination and transitions!).
I think ideally, all iframe content would only be loaded when it's on the current slide, then removed when that slide is no longer in action. But given how many codepen and youtube iframes I'm using, coming up with a one-size-fits-all solution could be tricky.
The text was updated successfully, but these errors were encountered: