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

Fastclick #803

Merged
merged 3 commits into from
Sep 17, 2013
Merged

Fastclick #803

merged 3 commits into from
Sep 17, 2013

Conversation

nicoburns
Copy link
Contributor

Issue #149 Closes #519

  • Uses FastClick to speed up click events instead of using Hammer.js tap
    events
  • Changes back button text from "Cancel" to "Back"
  • Removes box-shadow from back buttons.

Issue #149 Closes #519
- Uses FastClick to speed up click events instead of using Hammer.js tap
events
- Changed back button text from "Cancel" to "Back"
- Removed box-shadow from back buttons
@jgable
Copy link
Contributor

jgable commented Sep 17, 2013

Seems legit. 👍

@ErisDS
Copy link
Member

ErisDS commented Sep 17, 2013

Has failed the build

@nicoburns
Copy link
Contributor Author

The error appears to be:

FAIL core/client/mobile-interactions.js (1) core/client/mobile-interactions.js:8:5 'FastClick' was used before it was defined.

Is that just because I didn't put FastClick in the /*global */ comment at the top? (seeing as it works in the browser).

@javorszky
Copy link
Contributor

yes

@nicoburns
Copy link
Contributor Author

Above fixed the original fail. I've no idea what the new one is (I don't get it locally (although I do get two image uploading failures which don't seem to be related)).

EDIT: it now only fails in node 0.11. Is this acceptable?

@ErisDS
Copy link
Member

ErisDS commented Sep 17, 2013

The tests should pass, but Travis is being a bit shaky today. I have restarted them. You can also click on the link to restart any broken builds which you don't think should be broken.

ErisDS added a commit that referenced this pull request Sep 17, 2013
@ErisDS ErisDS merged commit a4aefb2 into TryGhost:master Sep 17, 2013
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.

Explore FastClick
4 participants