AmpliPi
AmpliPi copied to clipboard
Add sinewave animation component, begin implementation
What does this change intend to accomplish?
another attempt at closing #605, now with a react component animation that can be fit to purpose with just a bit of CSS work Using the slots param on the slider component
Checklist
- [ ] Have you tested your changes and ensured they work?
- [ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
- [ ] If applicable, have you updated the documentation/manual?
- [ ] If applicable, have you updated the CHANGELOG?
- [ ] Does your submission pass linting & tests? You can test on localhost using
./scripts/test - [ ] Have you written new tests for your core features/changes, as applicable?
- [ ] If this is a UI change, have you tested it across multiple browser platforms?
- [ ] If this is a UI change, have you tested across multiple viewport sizes (ie. desktop versus mobile)?
Original demo: https://github.com/user-attachments/assets/e78c03c4-d78c-41c9-9e3a-112bd0054b1b
New demo: https://github.com/user-attachments/assets/bd7ffd1a-4cd4-4a8a-96eb-2228f661f127
related to #759 as they both aim to solve the same issue, and both are going to be drafts for the foreseeable future until a meeting decides otherwise