I'd suggest that we remove it and rely on the much better variable translation support provided by variable.module/i18n. Exposing a locale group for 4 fixed strings IMHO unnecessarily clutters the UI.

A lot of the translation stuff now happens through translation sets and support for locale groups has actually been removed in 8.x.

CommentFileSizeAuthor
#1 locale_group.patch10.84 KBberdir

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new locale_group.patch10.84 KB

The attached patch removes the locale group stuff and replaces with a hook_variable_info() so that you can translate the variables.

Also added it to the README.txt.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Needs review » Fixed

Committed and pushed.

Worked perfectly. Note other (core) variables have descriptions however i couldn't find appropriate descriptions...

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Yeah, me neither :)

Status: Fixed » Closed (fixed)

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