Opened 5 years ago
Last modified 2 years ago
#50191 new enhancement
Propose https prefix, not http for external links
Reported by: | casiepa | Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | |
Component: | Editor | Keywords: | needs-patch |
Focuses: | ui-copy | Cc: |
Description
In https://build.trac.wordpress.org/browser/trunk/wp-includes/class-wp-editor.php?marks=1269#L1269
The text says: The URL you entered seems to be an external link. Do you want to add the required http:// prefix?
Might be better to propose to prefix 'https://' as default.
Change History (8)
This ticket was mentioned in PR #768 on WordPress/wordpress-develop by iamazik.
4 years ago
#3
- Keywords has-patch added; needs-patch removed
Update the default HTTP prefix to HTTPS as most sites now a days has SSL enabled and it's good for everything.
Trac ticket: https://core.trac.wordpress.org/ticket/50191
github-actions[bot] commented on PR #768:
4 years ago
#4
Hi @iamazik! 👋
Thank you for your contribution to WordPress! 💖
It looks like this is your first pull request to wordpress-develop
. Here are a few things to be aware of that may help you out!
No one monitors this repository for new pull requests. Pull requests must be attached to a Trac ticket to be considered for inclusion in WordPress Core. To attach a pull request to a Trac ticket, please include the ticket's full URL in your pull request description.
Pull requests are never merged on GitHub. The WordPress codebase continues to be managed through the SVN repository that this GitHub repository mirrors. Please feel free to open pull requests to work on any contribution you are making.
More information about how GitHub pull requests can be used to contribute to WordPress can be found in this blog post.
Please include automated tests. Including tests in your pull request is one way to help your patch be considered faster. To learn about WordPress' test suites, visit the Automated Testing page in the handbook.
If you have not had a chance, please review the Contribute with Code page in the WordPress Core Handbook.
The Developer Hub also documents the various coding standards that are followed:
- PHP Coding Standards
- CSS Coding Standards
- HTML Coding Standards
- JavaScript Coding Standards
- Accessibility Coding Standards
- Inline Documentation Standards
Thank you,
The WordPress Project
#6
follow-up:
↓ 7
@
2 years ago
- Keywords 2nd-opinion added
This one seems to be a bit more involved than just simply changing these two strings. Those strings are copied directly from the TinyMCE library and included here for translation purposes. I believe that changing them in class-wp-editor.php
without changing them in TinyMCE would result in those strings no longer being translated properly.
TinyMCE is currently on version 6.2, and WordPress includes 4.9.11 (last updated in [49557]). And far as I can tell, the 4.x branch is no longer maintained upstream, and we have not made any changes to the version of the library included in Core.
We could make changes to the bundled version on a case by case basis. But it would make it much more difficult to merge in an update from the maintainers should one become available (though unlikely). While this change does make sense (HTTPS is the protocol used on 80%+ of websites now), I'm not sure that it's worth the effort. Especially where WordPress no longer uses this library by default.
@azaozz Do you have any thoughts here?
#7
in reply to:
↑ 6
@
2 years ago
- Keywords needs-patch added; has-patch 2nd-opinion removed
Replying to desrosj:
Yes, the strings can be changed easily, but the function that adds the http://
prefix has to be updated too. That seems to be part of the link
plugin (if I'm not mistaken, it's been years...).
I'm also not sure it's worth the effort, unless it is a very easy/straightforward change.
Sure, good suggestion.