Added type=button to buttons

[mmcc]

Fixes #2470

[tjenkinson]

Don't you mean type=button?

On 13 Aug 2015, at 08:34, Matthew McClure notifications@github.com wrote:

Fixes #2470

You can view, comment on, or merge this pull request online at:

#2471

Commit Summary

added type=submit to button. fixes #2470
File Changes

M src/js/button.js (5)
Patch Links:

https://github.com/videojs/video.js/pull/2471.patch
https://github.com/videojs/video.js/pull/2471.diff
—
Reply to this email directly or view it on GitHub.

[pam]

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED

Commit: b40e8781f1f87976ed435744c8cd7dc7127ec3e2
Build details: https://travis-ci.org/pam/video.js/builds/75394605

(Please note that this is a fully automated comment.)

[mmcc]

Ha yes, yes I did mean that. Code is right, title is not. 

Edit: Updated. Thanks!

—
Sent from mobile

On Thu, Aug 13, 2015 at 12:41 AM, Project Automation Manager
notifications@github.com wrote:

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED
Commit: b40e8781f1f87976ed435744c8cd7dc7127ec3e2
Build details: https://travis-ci.org/pam/video.js/builds/75394605
(Please note that this is a fully automated comment.)

---

Reply to this email directly or view it on GitHub:
#2471 (comment)

[heff]

lgtm

[pam]

Tests failed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: BUSTED

Commit: efc9f70
Build details: https://travis-ci.org/pam/video.js/builds/75506086

(Please note that this is a fully automated comment.)