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

fixes #895 #1373

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@jackunion
Contributor

jackunion commented Jul 26, 2014

Moved event.preventDefault() to 'touchend' listener

jackunion added some commits Jul 26, 2014

fixes #895
Moved event.preventDefault() to 'touchend' listener
@gkatsev

This comment has been minimized.

Show comment
Hide comment
@gkatsev

gkatsev Jul 29, 2014

Member

LGTM

Member

gkatsev commented Jul 29, 2014

LGTM

@heff heff added confirmed labels Aug 1, 2014

@heff heff closed this in a0d8db9 Aug 4, 2014

@heff

This comment has been minimized.

Show comment
Hide comment
@heff

heff Aug 4, 2014

Member

Thanks for this! Hope you're doing ok.

Member

heff commented Aug 4, 2014

Thanks for this! Hope you're doing ok.

@jackunion

This comment has been minimized.

Show comment
Hide comment
@jackunion

jackunion Aug 5, 2014

Contributor

@heff my pleasure!

I'm fine, thank you. I'm about to start applying for jobs and, if I get lucky, for working visa in a couple of weeks. I hope that I'll lay my hands on videojs one day : ]

Take care!

Contributor

jackunion commented Aug 5, 2014

@heff my pleasure!

I'm fine, thank you. I'm about to start applying for jobs and, if I get lucky, for working visa in a couple of weeks. I hope that I'll lay my hands on videojs one day : ]

Take care!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment