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

Fix annotations example #2194

Closed
wants to merge 2 commits into from
Closed

Conversation

osheroff
Copy link
Contributor

Hi guys. Went to see if annotations was the solution to my problem.

It's not, but this fixes the public facing example.

… chrome

This fixes the region plugin as well as the annotations plugin.
@osheroff osheroff force-pushed the osheroff/fix_annotations_example branch from 2126981 to c795df8 Compare February 23, 2021 10:15
@osheroff
Copy link
Contributor Author

this also fixes the regions plugin example.

@thijstriemstra
Copy link
Contributor

It's not, but this fixes the public facing example.

What was specifically the issue? The build for this PR is now failing.

@osheroff
Copy link
Contributor Author

What was specifically the issue?

check out the examples for region and annotations (and anything that includes trivia.js); they're failing under chrome with "wavesurfer" is already defined.

The build for this PR is now failing.

yeah, seems unlikely that this PR caused it but I'll see.

@osheroff
Copy link
Contributor Author

babel/babel-eslint#487 is the same issue; class field declarations are throing a no-undef from eslint.

I've spent 10 minutes now and that's my yak shaving budget for the day. Yours if you want it :)

@thijstriemstra
Copy link
Contributor

fixed in #2196

@coveralls
Copy link

Coverage Status

Coverage remained the same at 82.407% when pulling e19e31f on osheroff/fix_annotations_example into 625c65a on master.

@katspaugh katspaugh deleted the osheroff/fix_annotations_example branch May 28, 2023 10:05
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.

3 participants