When translating persistent_login i noticed that the long strings didn't got picked up. This is because of the the use of "." to keep them within 80 letters. Example:

t('The maximum number of days for which a persistent '.
			 'login session is valid; afterwards, the user will '.
			 'need to log in again.  Enter 0 for no expiration.')

I'd suggest putting them on one line even if they are > 80 characters. I think that is what core does in most places. If we can agree on this I can prepare a patch.

CommentFileSizeAuthor
#1 persistent_login_t.patch4.91 KBanders.fajerson

Comments

anders.fajerson’s picture

Status: Active » Needs review
StatusFileSize
new4.91 KB

Why wait... Here is a patch.

bjaspan’s picture

Status: Needs review » Closed (fixed)

Fixed in D5 and HEAD.

anders.fajerson’s picture

Status: Closed (fixed) » Fixed

Lets mark it has fixed so it shows up in "My issues". Issues are autoclosed after a while. Thanks for the commit.

Anonymous’s picture

Status: Fixed » Closed (fixed)