svg-lib icon indicating copy to clipboard operation
svg-lib copied to clipboard

Add in keyboard shortcut to activate buttons

Open chookity-pokk opened this issue 1 year ago • 5 comments

This is some of the first actual elisp I've ever really written, so it might not be great. Though I did test this on my setup and made some buttons and was able to activate them using the keybinding I put in there. Which by default is unbound to anything. I also think I set it up in a way to make it so the user can edit the keybinding in their config.

chookity-pokk avatar Dec 30 '23 19:12 chookity-pokk

Looks pretty good actually. Is there a specific reason for the C-c C-s binding (instead of return or space for example) ? Also, there are some conflicts, you might need to rebase. And most importantly, since svg-lib is on ELPA, you'll need to sign the Copyright Assignment

rougier avatar Jan 09 '24 21:01 rougier

I can take care of the rebase after work today. I wasn't sure how to bind it to return while also still having return function normally, if that makes sense. If you know how then I will try implementing it!

I have signed a copyright assignment for the hyperbole project, so would I need to do it again for this project or does it apply to all contributions in general?

chookity-pokk avatar Jan 09 '24 21:01 chookity-pokk

@rougier Just an update here, I did a rebase and sent off for a copyright assignment. I haven't quite found a way to do the keyboard shortcut for enter/return without it overriding the whole buffer though.

chookity-pokk avatar Feb 05 '24 18:02 chookity-pokk

We could get inspiration from Emacs widget library and specifically the variable widget-keymap that is applied to widgets.

rougier avatar Feb 19 '24 09:02 rougier

I am currently dealing with a lot in my personal life so this likely won't be happening any time soon. If anyone wants to take this over, please feel free to. I will get to it when I have a chance but that likely won't be any time soon.

chookity-pokk avatar Feb 28 '24 21:02 chookity-pokk