Problem/Motivation

Examples UI capitalization should be consistent across modules.

This issue will be a meta for various types of work.

Proposed resolution

Module names should be capitalized, like "Content Entity Example". #2897096: Capitalize all module names

Route titles should be capitalized according to heirarchy. For instance, the top-level route "Form API Examples" should be fully capitalized, but child paths should be capitalized on the first word and then lower-case, like "Simple form" #2897630: Properly capitalize all route/link names

Toolbar titles should have the same title as the path they link. See "Stream Wrapper example", which should be fully-capitalized. Note: toolbar titles are derived from route titles, so no further action is needed after #2897630: Properly capitalize all route/link names

Remaining tasks

User interface changes

API changes

Data model changes

Comments

vegantriathlete created an issue. See original summary.

codypitts’s picture

Status: Active » Needs review
StatusFileSize
new4.91 KB

Changed capitalization for module names / menu links to more closely match the Core's.

Status: Needs review » Needs work

The last submitted patch, 2: inconsistent_capitalization-2831282.patch, failed testing.

vegantriathlete’s picture

@codypitts: Thanks for having a go at this!

You will want to look at *.info.yml, *.links.menu.yml and *.routing.yml. It also looks like you may need to update tests because they may be asserting certain text on the page.

However, before you get too far ahead of things I'd suggest you wait for input from the Module maintainer(s) about if / how they want to approach this.

mile23’s picture

Title: Inconsistent capitalization in module name / menu links » [meta] Inconsistent capitalization in module name / menu links
Issue summary: View changes
Status: Needs work » Active

This has been a bit on my mind for a while.

I think we should break this out into different categories of things for consistency.

Turning this into a meta and updating the IS.

vegantriathlete’s picture

Issue tags: +dcco2017

I'll queue this up for this weekend's DrupalCamp Colorado and see if we can get some people looking into it on Sunday.

mile23’s picture

Issue summary: View changes
mile23’s picture

Issue summary: View changes
michaellenahan’s picture

Issue tags: +Novice, +Vienna2017
joshmiller’s picture

Issue tags: -Vienna2017 +Nashville2018
vegantriathlete’s picture

Issue tags: -dcco2017

  • Mile23 committed 4f36484 on 8.x-1.x
    Issue #2831282 Capitalization cleanup.
    
alonaoneill’s picture

Status: Active » Reviewed & tested by the community

It looks like issue has been fixed and commited.
Marking RTBCed.
Thanks!

marvil07’s picture

Status: Reviewed & tested by the community » Fixed

Based on (a) @alonaoneill comment, (b) the related commit by @Mile23, and (c) a quick look at several places on the module as pointed by @vegantriathlete; I think it is OK to close this ticket.

I guess any new change would be better handled in a new ticket.

Thanks to all the people involved in this ticket!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.