Make WordPress Core

Opened 3 years ago

Closed 2 years ago

Last modified 2 years ago

#53028 closed defect (bug) (fixed)

Inconsistent Capitalisation

Reported by: presskopp's profile Presskopp Owned by: audrasjb's profile audrasjb
Milestone: 6.0 Priority: normal
Severity: normal Version:
Component: Text Changes Keywords: good-first-bug has-patch commit assigned-for-commit
Focuses: ui-copy Cc:

Description

just came across

documentation URL, documentation page, documentation links (lower case)

while mostly we use

Documentation URL, Documentation menu, Documentation link, Documentation dropdown etc (upper case)

Then there is "site URL", "website URL"

and probably more strings.

We should be consistent here.

Change History (20)

#1 @audrasjb
3 years ago

  • Owner set to audrasjb
  • Status changed from new to accepted

Good catch. I'm collecting the list of files right now, then I'll add the good-first-bug keyword :)

#2 follow-up: @audrasjb
3 years ago

  • Keywords good-first-bug added
  • Milestone changed from Awaiting Review to 5.8

Here are the occurrences I found in translatable strings.

Documentation page:

  • wp-admin/update-core.php

Documentation link:

  • wp-admin/option-discussion.php
  • wp-admin/option-writing.php

Documentation URL:
Unable to find it in translatable strings, only on translators comments.

Site/Website URL:

  • wp-admin/includes/class-wp-debug-data.php
  • wp-admin/includes/class-wp-privacy-policy-content.php
  • wp-admin/includes/ms.php

This ticket was mentioned in PR #1193 on WordPress/wordpress-develop by helloarpitgshah.


3 years ago
#3

  • Keywords has-patch added; needs-patch removed

#4 @arpitgshah
3 years ago

I just have made changes with each of the files as per the listing.

#5 in reply to: ↑ 2 @mukesh27
3 years ago

  • Keywords needs-refresh added

Hi there,

We should update those translators comments like in other files. Some files there are more.

meta-boxes.php#L705
options-general.php#L97

Replying to audrasjb:

Here are the occurrences I found in translatable strings.

Documentation URL:
Unable to find it in translatable strings, only on translators comments.

#6 @arpitgshah
3 years ago

Hey @mukesh27 & @audrasjb

Do we need to change this translator comments or string to capitalization?

Do we also need to add comments based on your list of path with all files or need to find and add from all the folders and file?

Please confirm.

Thanks,
Arpit G Shah

#7 @desrosj
3 years ago

Reviewing this, I'm not sure that the preferred change is to always capitalize "documentation". I personally wouldn't consider "documentation" to be a proper noun.

All instances of Documentation URL in Core that I can find are at the beginning of a sentence, and all instances of documentation URL are found within sentences, so these seem to be correct. Documentation page also has 6 lowercase instances and no capitalized, which also seems to be correct.

Below is my findings on the other strings:

  • Documentation link: 1 capitalized mid sentence, 7 lowercase
  • Documentation link(s): 1 capitalized mid sentence, 7 lowercase
  • Documentation menu: 1 capitalized mid sentence
  • Documentation dropdown: 1 capitalized mid sentence
  • Site URL: 3 incorrectly capitalized mid sentence, 54 correctly lower cased mid sentence.
  • Website URL: 2 instances lower cased as translation context.

The more common pattern that matches what I would expect to see is only capitalized when the first word of a sentence, label, or translation string/context, and lowercase everywhere else.

Today is the beta 1 deadline for 5.8. If someone is able to refresh the patch in the next ~6 hours, we can get this one in. But otherwise I will punt to 5.9 to allow more time.

#8 @arpitgshah
3 years ago

Hey @desrosj

Can you please provide the location of the files so that we can make it easy to work.

So as per your comments as we need to do beginning of the sentence will be capitalize and mid of sentence will be small cases.

Thanks,
Arpit G Shah

#9 @desrosj
3 years ago

@arpitgshah Sorry, I'm unable to give links to every instance in the code base right now. But performing a case sensitive search using your code editor should be able to uncover all of the instances for you.

#10 @audrasjb
3 years ago

  • Milestone changed from 5.8 to 5.9

Moving to 5.9 as it needs to be updated to handle other use cases.

#11 @sabernhardt
2 years ago

  • Focuses ui-copy added

#12 @audrasjb
2 years ago

  • Milestone changed from 5.9 to 6.0

This ticket still needs a refresh to cover other cases. Moving to 6.0 as 5.9 beta 1 is scheduled tomorrow.

This ticket was mentioned in Slack in #core by costdev. View the logs.


2 years ago

#14 @marybaum
2 years ago

AFter a cursory skim of the ticket, I'd break things down this way: leave it lower-case when talking about documentation as a concept, or a collective ("Please read the documentation!" and change to uppercase when we're talking about the Documentation Team ("For this release our Documentation Team reps suggested these people as leads.")

@davidbaumwald does that capture your distinction between proper and non-proper nouns?

(Seems to me improper nouns are the ones we don't say in family-friendly settings ;-D)

This ticket was mentioned in PR #2594 on WordPress/wordpress-develop by audrasjb.


2 years ago
#15

  • Keywords needs-refresh removed

#16 @audrasjb
2 years ago

The above PR addresses the few issues highlighted by @desrosj.

#17 @audrasjb
2 years ago

  • Keywords commit assigned-for-commit added

Tests are passing. Self assigning for commit consideration.

#18 @audrasjb
2 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 53186:

Text Changes: Fix various inconsistent capitalization issues.

Props Presskopp, arpitgshah, mukesh27, desrosj, audrasjb, marybaum.
Fixes #53028.

Note: See TracTickets for help on using tickets.