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

Use left/right arrow keys for changing to next/prev gallery slide #482

Merged
merged 45 commits into from
Nov 25, 2016

Conversation

vfonic
Copy link
Contributor

@vfonic vfonic commented Nov 16, 2016

No description provided.

ChrisBAshton and others added 15 commits May 16, 2016 14:22
a) casperjs isn't necessarily a dependency. Some people only use PhantomJS
b) this is causing issues when we try to upgrade to phantomjs 2.x, which requires casperjs beta 1.1. By listing casperjs as a dependency inside the Wraith gemspec, we're installing the default version of casperjs available to Rubygems, and this becomes the casperjs executable even if we've installed casperjs through something like brew. Consequently, we're stuck with an outdated version of casperjs and that prevents us from moving to phantomjs 2.

We should trust developers to sort out their own phantom/casper dependencies/configurations.
add casperjs as a developer dependency
Updated Dockerfile to install the new phantomjs and casperjs
Fix "undefined local variable or method `wraith'"
@ChrisBAshton ChrisBAshton changed the base branch from master to template_improvements November 25, 2016 15:31
@ChrisBAshton
Copy link
Contributor

Thanks @vfonic

@ChrisBAshton ChrisBAshton merged commit 1ff5d8c into bbc:template_improvements Nov 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants