AmpliPi icon indicating copy to clipboard operation
AmpliPi copied to clipboard

Add sinewave animation component, begin implementation

Open SteveMicroNova opened this issue 1 year ago • 2 comments

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)?

SteveMicroNova avatar Jul 18 '24 20:07 SteveMicroNova

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

SteveMicroNova avatar Jul 18 '24 20:07 SteveMicroNova

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

SteveMicroNova avatar Jul 18 '24 20:07 SteveMicroNova