- Issue created by @xjm
- Status changed to Needs review
3 months ago 6:56pm 18 June 2024 - Merge request !8448Fix the few places we are missing the hyphen for SDC's full name in core. β (Closed) created by xjm
- e0ipso Can Picafort
This is good for me. I didn't know about this grammatical rule, and it didn't come up earlier in any of the reviews.
My only question is about documentation. The status quo is that code and documentation are consistent bur wrong. Changing this in code is nice and easy, but now we have inconsistent documentation (official Drupal.org docs, video keynotes, DrupalCon presentations, DrupalCamp presentations, contrib project pages, slack messages, etc.)
I am OK with the change, and I will try to use the new for moving forward, if we agree on inconsistent docs. For the record, I think this is a minor issue, so I side with the inconsistency.
- Status changed to RTBC
3 months ago 8:12pm 18 June 2024 - πΊπΈUnited States xjm
Thanks @e0ipso. I had already tried to fix the handbook docs when I became aware they did not match MAINTAINERS.txt. It would more be things like videos and sessions from past content that would be slightly disrupted, and I think people can probably figure out that "Single Directory Components" in a slide deck refers to a thing that is officially called "Single-Directory Components".
The closest example I can think of where module names needed typographic modification was when they were incorrectly named (e.g.) "Content translation module" instead of "Content Translation module". I can't think of any equivalents where this specific issue of hyphenation came up for the naming of subsystems, but we have had to make a similar fix before in lots of other Drupal UI text strings, possibly field or element labels, etc.
- e0ipso Can Picafort
I think people can probably figure out that "Single Directory Components" in a slide deck refers to a thing that is officially called "Single-Directory Components".
Agreed. This is exactly what I meant when I said I think this is a minor issue, so I side with the inconsistency., but more eloquently put.
- πΊπΈUnited States xjm
Added an inverse example to the issue summary that is hopefully helpful for understanding this.
- πΊπΈUnited States xjm
Added a clearer inverse example to the issue summary that is hopefully helpful for understanding this.
- π«π·France nod_ Lille
Reviewed by a subsystem maintainer
Committed and pushed 1a487f3b7c to 11.x and 4074145ef7 to 11.0.x and 5601dc41ea to 10.4.x and acbd64f9b0 to 10.3.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.