I've noticed a few small bits of incorrect sentence casing. For reference, the User Interface Standards say: Page titles, links (in menus, too), and headings in Drupal should generally use sentence case capitalization, where only the first word is capitalized, except for proper nouns and other words that are generally capitalized by a more specific rule. Examples of sentence case capitalization include the following: "Block administration", "List menus", "Install new theme".

Most of Commerce does sentence casing just fine; these are just some small outliers I noticed. 1) On checkout/#/order_information, "Order information" is correct but "Order *S*ummary" needs improvement. 2) A few Views titles have casing that can be improved.

Screenshots attached.

Comments

Morbus Iff created an issue. See original summary.

morbus iff’s picture

Status: Active » Needs review
StatusFileSize
new5.76 KB
morbus iff’s picture

Status: Needs review » Reviewed & tested by the community
morbus iff’s picture

StatusFileSize
new7.06 KB

Patch updated for Commerce 2.24.

rszrama’s picture

Assigned: morbus iff » Unassigned

Easy +1 from me, just curious to know if we should worry about breaking translations or just go for it / provide a change record. Any idea on the process in other projects?

morbus iff’s picture

Easy +1 from me, just curious to know if we should worry about breaking translations or just go for it / provide a change record. Any idea on the process in other projects?

If a translator translated the strings without noticing they were wrong, that's on them ;)

I say break 'em. Dunno the process on other projects, though :(

morbus iff’s picture

Bump.

bojanz’s picture

+1 from me.

  • jsacksick committed fcce553 on 8.x-2.x authored by Morbus Iff
    Issue #3145772 by Morbus Iff, rszrama, bojanz: Incorrect sentence case...

  • jsacksick committed 02242d3 on 3.0.x authored by Morbus Iff
    Issue #3145772 by Morbus Iff, rszrama, bojanz: Incorrect sentence case...
jsacksick’s picture

Status: Reviewed & tested by the community » Fixed

Committed! Thanks!

Status: Fixed » Closed (fixed)

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