Follow-up to #1898466: update.module - Convert theme_ functions to Twig

We need to fix the lowercase strings in the parent case to be ucfirst per #421118: [Meta] Standardize capitalization on actions. These are: "ok", "warning", "error", maybe more

CommentFileSizeAuthor
#2 2287031-lcase-to-ucfirst-translatable-strings-2.patch1.17 KBl0ke

Comments

hass’s picture

hass commented
Status: Needs review » Active
l0ke’s picture

l0ke
Primary language English
Location Ukraine
commented
Status: Active » Needs review
StatusFileSize
new 2287031-lcase-to-ucfirst-translatable-strings-2.patch1.17 KB

I've checked the translatable strings and found out that only those you've noted aren't ucfirst.
Here is the patch.

hass’s picture

hass commented
Status: Needs review » Reviewed & tested by the community

Ok. Codewise perfect.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed b38632b and pushed to 8.x. Thanks!

  • alexpott committed b38632b on 8.x 
    Issue #2287031 by lokeoke | hass: Change lcase translatable strings to...

Status: Fixed » Closed (fixed)

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

m1r1k’s picture

m1r1k commented
Issue tags: +#ams2014contest