-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat equalizer #296
feat equalizer #296
Conversation
Awesome, the sound component itself could even be published as your own package since it exposes the same API as react-sound and could serve other people interested in replacing it. |
Good idea for the package, will check wich names are available. |
The equalizer itself could be created using the same sliders as the volume component, but vertically, or we could do something more fancy using chartjs, like this: https://cdn.guidingtech.com/media/assets/WordPress-Import/2016/04/apple-music-itunes-spotify-eq-equalizer-5.png |
You can also use .npmrc (and commit it if you want) https://codeburst.io/disabling-package-lock-json-6be662f5b97d?gi=c28afc04af04 |
i love the idea of chart.js |
As for chartjs, I've used it a lot at work, and created a little widget with it here: https://github.com/nukeop/react-ui-cards#cryptocurrency-card So we could reuse the part of config for those cards to make a fancy equalizer. Not sure if it's possible to drag the points on the chart easily though. |
everything is possible with chart.js ;) |
Where do you imagine the equalizer in the ui ?
|
I think it could occupy its own view, with a button to access it somewhere in the settings. |
do you have a screenshot of your work at the moment? can't wait to see it in action |
didnt notice i hav a little size problem i update this in 5 min |
looks really awesome |
For the presets we could:
|
We can also imagine some sound visualition as bar chart just behind the line chart |
And make the equalizer usage optional |
8088956
to
4513a36
Compare
4513a36
to
b1469f6
Compare
i force push because i rebase locally to resolve conflict before the big mess. sorry for the spam :) |
We could make the equalizer customizable by putting the frequencies used by the equalizer (and sound component) in the settings and make them editable. |
b1469f6
to
6882bd0
Compare
I have put the Sound component in another module: https://github.com/charjac/react-sound-html5. The last important thing to do is to replace dumb presets by reel ones. |
Awesome, fantastic work, as always. A good start would be to just copy some presets by eye from iTunes or whatever. If it ends up sounding off, we can correct it with another PR later. |
6882bd0
to
3b2a6f3
Compare
i add :
I put the equalizer link in the main menu, didnt get how to put it in settings. |
Ready for review and merging? |
yep |
Wow what an incredible contribution. Probably the best feature implemented so far, even among lots of fantastic quality code from all the contributors. I tested this and it all works great. There are only some minor edits to the UI that I'd like to add in the coming days. |
I replace react-sound by a react component with same props wrapping an audio tag.
I think next steps are: