-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 navigation with numbers #540
base: master
Are you sure you want to change the base?
Add navigation with numbers #540
Conversation
PR - jesseduffield#540 - Fix jesseduffield#522 - Like lazygit we can navigate with numbers. - jesseduffield/lazydocker@master...peauc:lazydocker:feature/issue-522-add-number-like-lazygit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one, just one comment
pkg/i18n/english.go
Outdated
FocusContainers string | ||
FocusImages string | ||
FocusVolumes string | ||
FocusNetwork string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FocusNetwork string | |
FocusNetworks string |
Should be plural for consistency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jesseduffield changes have been made :)
Looks good @peauc. Can you squash your commits down to one? |
@jesseduffield of course, I do not mind. |
7ead397
to
1dc8033
Compare
A la lazygit, we can now navigate between panels using numbers (1,2,3,4,5,6,7) Add translation for number navigation reformat oops fix translation case & generate docs pluralize networks
1dc8033
to
86cce84
Compare
alright its done :) @jesseduffield |
PR - jesseduffield#540 - Fix jesseduffield#522 - Like lazygit we can navigate with numbers. - jesseduffield/lazydocker@master...peauc:lazydocker:feature/issue-522-add-number-like-lazygit
Small nudge @jesseduffield |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fix #522
Like lazygit we can navigate with numbers.