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 leading space to Kubernetes context suggestions #2791

Merged
merged 2 commits into from
Aug 18, 2024

Conversation

uozalp
Copy link
Contributor

@uozalp uozalp commented Jul 12, 2024

This pull request addresses an issue with the current behavior of ctx suggestions. Previously, when users typed ctx, the first context suggestion would appear immediately adjacent to ctx without a leading space (e.g., ctx<context>), making it not possible to select the suggestion by pressing enter. This behavior required users to manually add a space before the context name, which was not intuitive and disrupted the workflow.

The proposed change introduces a leading space to the context suggestions. Now, when ctx is typed, the suggestion will appear as ctx <context>, allowing users to seamlessly use the arrow keys to navigate through the context suggestions and select one by pressing enter. This enhancement streamlines the user experience by making context selection more intuitive and efficient.

@KevinGimbel
Copy link

LGTM! Tested the PR and its working as expected.

@derailed I think this can be merged with the next release.

Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@uozalp Great catch! Thank you!

@derailed derailed added the bug Something isn't working label Aug 15, 2024
@derailed derailed merged commit 88394a4 into derailed:master Aug 18, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants