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 navigation with numbers #540

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

peauc
Copy link

@peauc peauc commented Jun 5, 2024

Fix #522
Like lazygit we can navigate with numbers.

jorgemxm added a commit to jorgemxm/lazydocker that referenced this pull request Jun 12, 2024
Copy link
Owner

@jesseduffield jesseduffield left a 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

FocusContainers string
FocusImages string
FocusVolumes string
FocusNetwork string
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
FocusNetwork string
FocusNetworks string

Should be plural for consistency

Copy link
Author

@peauc peauc Jul 26, 2024

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 :)

@jesseduffield
Copy link
Owner

Looks good @peauc. Can you squash your commits down to one?

@peauc
Copy link
Author

peauc commented Jul 26, 2024

@jesseduffield of course, I do not mind.

@peauc peauc force-pushed the feature/issue-522-add-number-like-lazygit branch from 7ead397 to 1dc8033 Compare July 26, 2024 13:57
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
@peauc peauc force-pushed the feature/issue-522-add-number-like-lazygit branch from 1dc8033 to 86cce84 Compare July 26, 2024 13:58
@peauc
Copy link
Author

peauc commented Jul 26, 2024

alright its done :) @jesseduffield

jorgemxm added a commit to jorgemxm/lazydocker that referenced this pull request Jul 27, 2024
@peauc
Copy link
Author

peauc commented Sep 20, 2024

Small nudge @jesseduffield

Copy link
Owner

@jesseduffield jesseduffield left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Move the panel with number like lazygit
2 participants