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

Do not allow deleting current client from GUI #677

Open
searinminecraft opened this issue Aug 4, 2024 · 1 comment
Open

Do not allow deleting current client from GUI #677

searinminecraft opened this issue Aug 4, 2024 · 1 comment
Labels
a:feature New feature or request

Comments

@searinminecraft
Copy link

searinminecraft commented Aug 4, 2024

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

You could accidentally delete your current client from within the GUI, logging you out in the process

Describe the solution you'd like
A clear and concise description of what you want to happen.

There should be an indication that its your current client, and delete/hide the delete button.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

None

Additional context
Add any other context or screenshots about the feature request here.

Concept:

image

@searinminecraft searinminecraft added the a:feature New feature or request label Aug 4, 2024
@jmattheis
Copy link
Member

I don't think we have to disable the delete button but highlighting the current session seems reasonable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:feature New feature or request
Development

No branches or pull requests

2 participants