#50886 closed defect (bug) (fixed)
docs: Improve help text about site vs. WordPress URL settings
Reported by: | tobifjellner | Owned by: | marybaum |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Help/About | Keywords: | has-patch commit |
Focuses: | ui, docs, administration, ui-copy | Cc: |
Description
Hi,
I'm looking at https://build.trac.wordpress.org/browser/trunk/wp-admin/options-general.php?marks=30#L30
and thinking about how to improve the help string:
"The WordPress URL and the Site URL can be the same (example.com) or different; for example, having the WordPress core files (example.com/wordpress) in a subdirectory instead of the root directory."
- Since the URL's should include http/https, I believe the sample should also have that: "https://example.com)
- It would be good to elaborate a bit more on the difference between these two URL's. What about "WordPress URL is where the core files of WordPress are installed. Site URL is the URL a visitor enters in their browser in order to reach your site."
- Perhaps we should also mention here about http/https specifically?
- What about: Remember that if you want to move your site to a different domain, you'll usually need to first configure this outside of WordPress.
Attachments (4)
Change History (54)
#2
in reply to:
↑ description
@
4 years ago
This ticket was mentioned in Slack in #core by abhanonstopnews. View the logs.
3 years ago
This ticket was mentioned in Slack in #core by docpop. View the logs.
3 years ago
#8
@
3 years ago
Just added a patch with new copy:
Explains the difference between site and WP URLs.
Changes passive verbs to active.
Addresses http versus https.
Please feel free to make cuts!
This ticket was mentioned in Slack in #core by marybaum. View the logs.
3 years ago
This ticket was mentioned in Slack in #core by abhanonstopnews. View the logs.
3 years ago
#12
@
3 years ago
Looks like this patch could use some review and testing. I will test the patch and, if it works properly (it's copy ... so I'm optimistic) then I'll ask for review toward commit in this week's devchat.
#13
@
3 years ago
Discussed in bug scrub. Text in @marybaum's patch reviewed. A new patch will be needed to incorporate the other changes to text and @marybaum's text changes too. @snoozeulose to look at the wording and draft a patch. Thanks to all those who came to discuss the ticket.
This ticket was mentioned in Slack in #core by flexseth. View the logs.
3 years ago
This ticket was mentioned in Slack in #core by marybaum. View the logs.
2 years ago
#17
follow-up:
↓ 19
@
2 years ago
- Keywords changes-requested added
I think the part of the patch that is replacing can't
with cannot
in QUnit tests comes from another patch, and shouldn't be there :)
#19
in reply to:
↑ 17
@
2 years ago
Replying to audrasjb:
I think the part of the patch that is replacing
can't
withcannot
in QUnit tests comes from another patch, and shouldn't be there :)
Yes, there are quite a few unrelated changes, mostly from [52978], [53131], [53182], and their earlier iterations. If these suggestions were properly uploaded and discusssed on #38913 instead of Slack at the time, they would have been much easier to review :)
I was able to refresh the patch by:
- Checking out an earlier revision from the date the patch was submitted, i.e. [52111].
- Applying the patch.
- Resolving merge conflicts manually and trying to separate meaningful edits from unrelated changes.
50886.diff should only include legitimate copy edits (apologies if I missed some when resolving the conflicts), though most are still unrelated to this ticket and should be moved to a new one to avoid further confusion. Looks like some were meant for #38913 but never made it to core due to not being uploaded to the ticket, others are just various edits using simpler language or replacing passive voice with active.
The changes relevant to this ticket are in wp-admin/options-general.php
. I have not reviewed them yet, it should be easier once the other changes are moved to a separate ticket.
@
2 years ago
Removed all the changes that don't belong to the specific text of this ticket and applied the patch to a local install.
This ticket was mentioned in Slack in #core by marybaum. View the logs.
2 years ago
#23
follow-up:
↓ 24
@
2 years ago
- Milestone changed from 6.1 to 6.2
Unless a committer wants to add this within the next week, let's revisit the ticket in the next release cycle.
In addition to the typo fix, I have a few suggestions:
- The padlock is not always to the left of the address. Right-to-left language translators could correct that to the right, but "next to" would fit either language direction.
- Wrapping the URL examples and HTTP(S) protocols with
code
tags can help as well. The text string with protocols could usesprintf
with placeholders. The URL example should be translatable (French hasexemple.fr
), so that might involve adding thecode
tags within the string. - I don't like the "reach your objectives" phrase, mainly because building trust is a fundamental objective in itself (without visitors' trust, how can anyone sell, inform, persuade, etc.?). One simple option is to end the sentence at "search engines." Another option is to combine the first part of that sentence with the second part of the previous sentence: "Both of those things signal to visitors that your site meets some basic security requirements, which can build trust with your users and with the search engines."
#24
in reply to:
↑ 23
@
2 years ago
Replying to sabernhardt:
I agree let's move this to 6.2.
On your other points:
- Yes let's change the text to 'next to'
- A slight tweak of both of your text to:
"Both of those things signal to visitors that your site meets some basic security requirements, which can build trust with your users and with search engines."
@marybaum over to you to make this change in 2) and 1) and 3), and remove the space in the url (comment 21)?
Thanks @sabernhardt and @marybaum .
Unless a committer wants to add this within the next week, let's revisit the ticket in the next release cycle.
In addition to the typo fix, I have a few suggestions:
- The padlock is not always to the left of the address. Right-to-left language translators could correct that to the right, but "next to" would fit either language direction.
- Wrapping the URL examples and HTTP(S) protocols with
code
tags can help as well. The text string with protocols could usesprintf
with placeholders. The URL example should be translatable (French hasexemple.fr
), so that might involve adding thecode
tags within the string.- I don't like the "reach your objectives" phrase, mainly because building trust is a fundamental objective in itself (without visitors' trust, how can anyone sell, inform, persuade, etc.?). One simple option is to end the sentence at "search engines." Another option is to combine the first part of that sentence with the second part of the previous sentence: "Both of those things signal to visitors that your site meets some basic security requirements, which can build trust with your users and with the search engines."
This ticket was mentioned in Slack in #core by abhanonstopnews. View the logs.
2 years ago
@
2 years ago
A new patch in response to comment 24 discussion, and a thought I had about the basic security copy.
#28
@
2 years ago
- Keywords needs-testing added; changes-requested removed
- Owner set to marybaum
- Status changed from new to assigned
#30
@
22 months ago
Patch looks good at the moment.
I have two suggestions. If intentional then we can keep as it is too.
Patch: https://core.trac.wordpress.org/attachment/ticket/50886/50886-comment%2024.patch
Proposing:
The WordPress URL is the where the core WordPress installation files are, and the site URL is the address a visitor uses in the browser to get go to your site.
Changes to make:
- Remove
the
- Replace
get
withgo
This ticket was mentioned in PR #4064 on WordPress/wordpress-develop by @sabernhardt.
22 months ago
#31
Combines patches from Trac 50886 for General Settings page
#32
@
22 months ago
Proposed new text (for single sites):
Most themes show the site title at the top of every page, in the title bar of the browser, and as the identifying name for syndicated feeds. Many themes also show the tagline.
Two terms you will want to know are the WordPress URL and the site URL. The WordPress URL is where the core WordPress installation files are, and the site URL is the address a visitor uses in the browser to go to your site.
Though the terms refer to two different concepts, in practice, they can be the same address or different. For example, you can have the WordPress core files in the root directory (
https://example.com
), in which case the two URLs would be the same. Or the WordPress files can be in a subdirectory (https://example.com/wordpress
). In that case, the site URL and the WordPress URL would be different.
Both site URLs and WordPress URLs can start with either
http://
orhttps://
. A URL starting withhttps://
will also show a padlock next to the address in the browser address bar. Both of those things signal to visitors that your site meets some basic security requirements, which can build trust with your users and with search engines.
If you want site visitors to be able to register themselves, check the membership box. If you want the site administrator to register every new user, leave the box unchecked. In both cases, you can set a default user role for all new users, whether they register themselves or a site admin registers them.
You can set the language, and WordPress will automatically download and install the translation files (available if your filesystem is writable).
Notes:
- The PR includes
code
tags. - In addition to explaining Site URLs and WordPress URL, this includes changing two strings to replace passive voice. One of these strings is also in network settings.
- I had tried splitting the reminder to save into its own text string so the old translations would remain valid. But then I removed that part from the patch because I'm not comfortable editing it in six files for this ticket. It could deserve a separate discussion.
- 50886-comment 24.patch proposed "Both of those things signal to visitors that your site is authentic, and the people who run it are real." However, I prefer the sentence that talks about building trust.
@audrasjb commented on PR #4064:
22 months ago
#33
The patch looks good to me. Only wondering whether it would be useful to point people to this HelpHub link?
@sabernhardt commented on PR #4064:
22 months ago
#34
The HelpHub link was already directly below the Site Address field, but it's probably worth including in both places.
@audrasjb commented on PR #4064:
22 months ago
#35
Ah you're right. In that case, I'm not sure it's worth having it on both locations
@sabernhardt commented on PR #4064:
22 months ago
#36
I updated the documentation URL. I'm more inclined to add the link, even if people might ignore it, in case someone does want it there. If the link should be removed, though, reverting the last two commits would be easy enough.
@audrasjb commented on PR #4064:
22 months ago
#37
Thinking about it, I think we should put that link back in the help tabs anyway, because it's useful to have it in that context too 👍
#38
@
21 months ago
@costdev suggested a few changes to the PR:
- Consistently mentioning WordPress URL before site URL.
- Reusing "core WordPress installation files" instead of "WordPress core files": 'For example, you can have the core WordPress installation files in the root directory (
https://example.com
), in which case the two URLs would be the same.' - Reducing repetition in the default user role sentence (and I edited "In both cases" to "In either case"): 'In either case, you can set a default user role for all new users.'
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
21 months ago
#40
@
21 months ago
- Keywords commit added; needs-testing removed
This ticket was discussed during the bug scrub. As feedback on the PR has been addressed, I think this one is ready for commit
consideration.
Adding the keyword and pinging @audrasjb to take a look regarding this comment on the PR - does this still need to be done or?
Additional props: @mukesh27
#41
@
21 months ago
Both WordPress URLs and site URLs can start with either
http://
orhttps://
. A URL starting withhttps://
will also show a padlock next to the address in the browser address bar. Both of those things signal to visitors that your site meets some basic security requirements, which can build trust with your users and with search engines.
I think this part can be confusing, as it does not mention that https://
only works if you set up an SSL certificate, or if the hosting provider does that for you. I've seen people on support forums with this exact issue, changing these URLs to https://
before setting up SSL and then asking why the site can no longer be opened.
The suggested text seems to encourage changing the URLs to https://
without explaining the consequences.
Should we try to clarify that here?
21 months ago
#43
@SergeyBiryukov How does this sound?
Both WordPress URLs and site URLs can start with either
http://
orhttps://
. A URL starting withhttps://
requires an SSL certificate, so be sure that you have one before changing tohttps://
. Withhttps://
, a padlock will appear next to the address in the browser address bar. Bothhttps://
and the padlock signal to visitors that your site meets some basic security requirements, which can build trust with search engines and your visitors.
@audrasjb commented on PR #4064:
21 months ago
#44
Sounds great to me 👍
#45
follow-up:
↓ 47
@
21 months ago
@SergeyBiryukov @audrasjb With the feedback on the PR addressed, do you think this one can be committed before the 6.2 Beta 4 release party as we're entering soft-string freeze?
@sabernhardt commented on PR #4064:
21 months ago
#46
The proposed text is good, but I specifically want visitors before search engines in the last sentence.
I updated the PR in case it's worth committing this late in the cycle. However, with soft string freeze today, this ticket probably should be moved to 6.3.
#47
in reply to:
↑ 45
@
21 months ago
- Keywords commit added; changes-requested removed
Replying to costdev:
With the feedback on the PR addressed, do you think this one can be committed before the 6.2 Beta 4 release party as we're entering soft-string freeze?
Yes, looks good to me now :)
This ticket was mentioned in Slack in #core by costdev. View the logs.
21 months ago
@SergeyBiryukov commented on PR #4064:
21 months ago
#50
Thanks for the PR! Merged in r55452.
Replying to tobifjellner:
I agree on point 1, about adding http/https to the URL. But I'm wondering, should we include both? The https://example.com and http://example.com?
Yeah, it makes sense IMO to rephrase the original sentence. @marybaum, I'd love to have your feedback about the proposed wording :).
What if we include instead a link to a resource that clarify that. We don't want the help panel to be also very long I think.
+1