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
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | inconsistent_capitalization-2831282.patch | 4.91 KB | codypitts |
Comments
Comment #2
codypitts commentedChanged capitalization for module names / menu links to more closely match the Core's.
Comment #4
vegantriathlete@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.
Comment #5
mile23This 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.
Comment #6
vegantriathleteI'll queue this up for this weekend's DrupalCamp Colorado and see if we can get some people looking into it on Sunday.
Comment #7
mile23Comment #8
mile23Comment #9
michaellenahan commentedComment #10
joshmillerComment #11
vegantriathleteComment #13
alonaoneill commentedIt looks like issue has been fixed and commited.
Marking RTBCed.
Thanks!
Comment #14
marvil07 commentedBased 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!