buttondown / roadmap

Buttondown's public roadmap
53 stars 0 forks source link

Multiple "cmd+I to copy" commands fire at once #3009

Closed benborgers closed 2 months ago

benborgers commented 3 months ago

When I hit cmd+I on a subscriber page, everything copies at once. Ideally only the hovered one (?) should trigger.

2024-07-04 at 09 44 11@2x

mary-ext commented 2 months ago

I'm guessing this is Mac-specific because that Copy shortcut doesn't seem to work for me and after taking a look, it appears that we're binding cmd+i on mount and unbinding ctrl+i on unmount, not sure if that's relevant but just putting it in for now

mary-ext commented 2 months ago

Aha, this seems to be stemming from the fact that we also have 3 copiable component instances in the subscriber details drawer. I'm going to prioritize the email first over anything else.