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

Ports from es5: Part two #1193

Merged
merged 9 commits into from
Aug 20, 2017
Merged

Ports from es5: Part two #1193

merged 9 commits into from
Aug 20, 2017

Conversation

mspae
Copy link
Contributor

@mspae mspae commented Aug 19, 2017

Adds features and fixes that were added to the es5 branch (previously master) to the new master (previously next).

See #1092

First part: #1191

quentinhayot and others added 9 commits August 19, 2017 11:11
I removed the call to `.empty()`when handling the resize of the container when `responsive`is set to `true`.
This allow the playback to keep going while the container is resizing.
Should not close audiocontext when it's already closed. Chrome throws a warning if done so.
* add barHeight parameter

* Update drawer.multicanvas.js

* Update drawer.canvas.js
* add getFilters method

* add getFilters() method

add 'Filters must be set with setFilters method first' note.

* return empty array if no filters are set

* change to `return this.backend.filters || [];`
This was referenced Aug 19, 2017
@mspae
Copy link
Contributor Author

mspae commented Aug 20, 2017

BTW 4ee7aab fixes a bug that was introduced in the first half (incorrect implementation of util.frame in drawWave and drawBars caused the waveform to be empty) so this should be merged ASAP.

@katspaugh
Copy link
Owner

Great, thanks!

@katspaugh katspaugh self-requested a review August 20, 2017 15:11
@mspae mspae merged commit 2966471 into katspaugh:master Aug 20, 2017
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.

None yet

6 participants