Navigation Menu

Skip to content
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

Add Ctrl+g and G vim style keybindings to move cursor to first and last line #141

Closed
wants to merge 2 commits into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Nov 6, 2019

I add new keybindings for cursor motion.

ctrl+g and G are Vim-like keybinding that move cursor to first and last line of the list. Real vim keybinding to move cursor to first line is gg but it seems that keybindings that use two or more key are not supported in this app so I assign the fiunction to ctrl+g.

I think that i implemented them in all blocks (Artists, Albums, Playlists and so on) but if you notice my over looking, please tell me it.

Copy link
Owner

@Rigellute Rigellute left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job working on this.

I can't tell whether you've missed anything as I can no longer grok the way I've layed out the selection indexes for all the different views - it's quite complicated!

This motiviates me to try and simplify things for the future.

}
}

pub fn go_last_line(app: &mut App) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eventually, we could probably improve things by having a more consistent way of managing the selection index. I'll look into this as part of working on better pagination.

@ghost ghost closed this Nov 6, 2019
@ghost ghost deleted the vim-style-keybindings branch November 8, 2019 06:49
This pull request was closed.
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.

None yet

1 participant