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

Update placeholder to https:// #2323

Merged
merged 1 commit into from Nov 27, 2017

Conversation

Projects
None yet
4 participants
@flokX
Contributor

flokX commented Nov 26, 2017

and motivate users to use the secure version of the url.

Update placeholder to https://
and motivate users to use the secure version of the url.

@ozh ozh closed this Nov 26, 2017

@ozh

This comment has been minimized.

Show comment
Hide comment
@ozh

ozh Nov 26, 2017

Member

This is the input box where people enter the URL to shorten.

Member

ozh commented Nov 26, 2017

This is the input box where people enter the URL to shorten.

@dgw

This comment has been minimized.

Show comment
Hide comment
@dgw

dgw Nov 27, 2017

Collaborator

@ozh Yes, and I actually think it's a reasonable place to show https:// as a subtle way to maybe get users to check if their link is available over HTTPS if it isn't already the secured version.

Collaborator

dgw commented Nov 27, 2017

@ozh Yes, and I actually think it's a reasonable place to show https:// as a subtle way to maybe get users to check if their link is available over HTTPS if it isn't already the secured version.

@LeoColomb

This comment has been minimized.

Show comment
Hide comment
@LeoColomb

LeoColomb Nov 27, 2017

Member

I second you @dgw. I think also it's worth adding it.

Member

LeoColomb commented Nov 27, 2017

I second you @dgw. I think also it's worth adding it.

@ozh ozh reopened this Nov 27, 2017

@ozh ozh merged commit 84efce5 into YOURLS:master Nov 27, 2017

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Scrutinizer No new issues
Details
codacy/pr Good work! A positive pull request.
Details
@ozh

This comment has been minimized.

Show comment
Hide comment
@ozh

ozh Nov 27, 2017

Member

The people have spoken!

Member

ozh commented Nov 27, 2017

The people have spoken!

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