Skip to content

Add visual feedback to bindings#14

Merged
prgmitchell merged 1 commit intoprgmitchell:mainfrom
xal3xhx:feat/binding-trigger-feedback
Apr 18, 2026
Merged

Add visual feedback to bindings#14
prgmitchell merged 1 commit intoprgmitchell:mainfrom
xal3xhx:feat/binding-trigger-feedback

Conversation

@xal3xhx
Copy link
Copy Markdown
Contributor

@xal3xhx xal3xhx commented Apr 17, 2026

Summary

  • Binding items in the list now pulse with an indigo glow when triggered by MIDI
  • Glow stays active while events stream in (debounced 300ms past last event)
  • Fades in/out via CSS box-shadow transition, no looping pulse

Implementation

  • src/features/bindings/bindings.js — set data-binding-id on each binding item root
  • src/main.jsflashBindingTrigger(id) helper called from the midi_event listener once a binding is resolved; stashes timeout on the element and clears/resets on each incoming event
  • src/styles/bindings-and-panels.css.binding-item.triggered sets a static glow; base selector has transition: box-shadow 350ms ease-out for the fade
  • Dark-mode variant uses a lighter indigo (rgba(129, 140, 248, ...))

Test plan

  • Move a fader bound to a volume target — binding pulses smoothly while moving, fades out ~300ms after stopping
  • Press a button bound to ToggleMute / Hotkey / OpenApplication — single pulse visible
  • Trigger bindings with integration targets (OBS / WaveLink) — pulse visible
  • Toggle dark mode — glow color matches theme
  • Rapid input on multiple bindings at once — each pulses independently

Pulse glow appears on binding items when triggered by MIDI. Stays active
while events stream in (debounced 300ms) and fades out via CSS transition.
@prgmitchell prgmitchell merged commit b4862ad into prgmitchell:main Apr 18, 2026
1 check passed
@xal3xhx xal3xhx deleted the feat/binding-trigger-feedback branch April 18, 2026 18:01
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.

2 participants