-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEEDBACK] Implement buffer history for alt+a and alt+<, alt+>, and alt+/ #449
base: master
Are you sure you want to change the base?
Conversation
7539b9e
to
b1ca4b6
Compare
the keyboard shortcut part of this needs a lot of testing, that stuff sucks :( I can't get meaningful events for alt+< and alt+> with QWERTZ at all, and with Neo2, alt+> conflicts with alt+g because < is mod3+g in neo :\ |
What do you guys think about the direction this is heading in? |
b1ca4b6
to
b309cf2
Compare
Comments on this? |
77a2c47
to
c83669b
Compare
@lorenzhs Looks good to me. Cannot really test ALT-> ALT-< with my QWERTY though. I've made some minor changes to the code. You can see them in my commits, and i'll let you merge the branch if you think that they're good. |
e967441
to
89bd2b2
Compare
I can't use Alt+> with Neo2 either, it triggers Alt+G (because > is Mod3+G in neo, where Mod3 is Caps Lock :D) |
the last two commits look good, but what's up with the old stuff that got mixed up in here? |
89bd2b2
to
3108295
Compare
I might have rebased on top of an outdated master ;-) It's fixed now. |
@lorenzhs what do you think of this? should we merge? |
3108295
to
f7041c9
Compare
is there any interest in reviving this? |
Please merge? |
Fixes #360 and adds a couple of new features :)