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

Split pasted host with port into the separate input fields #455

Closed
vvondra opened this Issue Nov 15, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@vvondra

vvondra commented Nov 15, 2017

Very often you copy a hostname with concatenated hostname and port, such as mydatabase.com:5432

If you post this into the host field, it will fail to connect and often the error is not obvious since the hostname is long and the port is hidden on the right.

It would be a nice UX touchup that if the pasted value contains a port, it would be automatically placed into the correct input field.

@jakob

This comment has been minimized.

Owner

jakob commented Nov 15, 2017

Good point. We could probably extend that to other strings (eg. URLs), so that you can paste a connection URL and it'll parse it and put everything in the right fields.

@jakob

This comment has been minimized.

Owner

jakob commented Jan 12, 2018

I've now added automatic parsing of hostname:port and URLs to Postico.

Can you download build 2235 and see if it works as expected?
https://eggerapps-downloads.s3.eu-west-1.amazonaws.com/postico-2235.zip

@vvondra

This comment has been minimized.

vvondra commented Jan 12, 2018

Works perfectly!

@jakob

This comment has been minimized.

Owner

jakob commented Jan 15, 2018

I've tweaked the behavior slightly so that it only triggers the parsing when you are at the end of the text field.

This should hopefully prevent accidental triggers of this feature while editing IPv6 addresses etc.

https://eggerapps-downloads.s3.eu-west-1.amazonaws.com/postico-2239.zip

@jakob jakob added the done label Jan 15, 2018

@postico-bot

This comment has been minimized.

postico-bot commented Jan 23, 2018

We have been working on this issue!

jakob: Allow pasting Redshift URLs into the host field #455

Download Build B2293, or check out the list of recent builds.

@jakob jakob closed this Mar 29, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment