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

Prevent saving of double connections #45

Open
tobimori opened this issue Apr 23, 2020 · 2 comments
Open

Prevent saving of double connections #45

tobimori opened this issue Apr 23, 2020 · 2 comments
Labels
bug Something isn't working low issues with a low priority

Comments

@tobimori
Copy link
Collaborator

grafik

@tobimori tobimori added bug Something isn't working low issues with a low priority labels Apr 23, 2020
@matthiaaas
Copy link
Owner

Is this necessary? Not sure if you should block this. Because to save multiple connections you have to reload the view so the button becomes enabled again.
Probably there are use cases for double saves.

@tobimori
Copy link
Collaborator Author

tobimori commented Apr 23, 2020

@matthiaaas Why there should be a use case? You save paths per user/host, not mehr QuickConnect connection and you can't jump between two of the same connections with QuickConnect.
grafik

I'd replace the save connection icon with a filled out icon to unsave the server when all details are the same, and save it with an unfilled icon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working low issues with a low priority
Projects
None yet
Development

No branches or pull requests

2 participants