fix(microphone): stop hold-to-talk ping ring from stealing clicks - #80
Conversation
Signed-off-by: Robbin "Roboroads" Schepers <robbin@deschepers.nl>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ping overlay span shown when the microphone is live inside the hold-to-talk button was updated to add the pointer-events-none class and reformatted across multiple lines, preventing it from intercepting pointer clicks. ChangesPing overlay pointer-events fix
Estimated code review effort: 1 (Trivial) | ~2 minutes Related IssuesFixes: Suggested labels: bug, ui Suggested reviewers: N/A Poem 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Adds pointer-events-none to the pulsing ring span so its animate-ping-expanded hit area no longer intercepts clicks meant for controls above the hold-to-talk button.
Fixes #70
Summary by CodeRabbit