Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Some of our users don't understand what "Remember Me" means (I expect it is because "remember me" is often used to save login info like a username or password). We'd like to tweak the text ("Keep me logged in on this computer") but can't do so easily. It'd be nice to have a configurable option for this within the module.
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | configurable_text_on-2763523-15.patch | 4.44 KB | nitesh sethia |
| #12 | remember-me-japanese.PNG | 7.15 KB | kapil.ropalekar |
| #12 | remember-me-france.PNG | 10.56 KB | kapil.ropalekar |
Comments
Comment #2
gappleThat is a really good suggestion.
I'll add it in to the 8.x version, since translatable configuration is really simple, and hopefully someone can help with a patch for 7.x
Comment #4
gappleCommitted to 8.x, so setting the issue back to 7.x
Comment #5
jalpesh commentedI have try to add same configuration for drupal 7 module.
Let me know my mistake in patch. Thanks.
Comment #7
jalpesh commentedHere we go with new patch, change in label name to make consistence D8 module label.
Comment #9
gappleThanks for the patch. It's a good start, but
t()should only be used for translating static strings.It's been a while since I've worked with a multilingual site, so I'll have to look more into the right way to enable translation of user-configurable interface text.
Comment #10
jalpesh commentedYup, you are right. we should not use t() tag to translating variable as mention in https://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/t/7.x. I think the patch is failing because of following line in patch.
'#title' => t(variable_get('persistent_login_remember_label', PERSISTENT_LOGIN_REMEMBER_LABEL))
I will try to find out other solution. Thanks.
Comment #11
gappleThe testing failure was just due to there not yet being any tests to run, so I've disabled automated tests.
Comment #12
kapil.ropalekar commentedHi gapple,
I have created a patch and tested the "Remember me" text to be translatable on a multi_lingual instance.
The configuration text-field for "Remember me" under admin/config/system/persistent_login has been added as Form Label. I feel that we should keep this field required so that blank value validation will be taken care of.
Also added the variable_del under the uninstall hook.
Attached are a few screenshots for your reference where i tired testing the "Remember me" text on the user login page for the Languages Japanese and France.
Could you kindly test the patch and let me know your reviews.
Thanks !
Comment #13
jalpesh commentedThank @kapil.ropalekar for patch.
I hope that you have also take care of https://www.drupal.org/node/2766223 in your patch as it was already committed to 8.x branch.
Comment #14
archimedes commentedThanks for the work, everyone! Any more movement on this? What's next? I'd love to get this into our site soon.
Comment #15
nitesh sethia commentedFixing the Indentation issue and Moving the form label change configuration of the top of the configuration page.
Comment #16
gappleNone of the patches have addressed that
t()should only be used for translating static strings, as was mentioned in comment #9Comment #17
somnolentsurfer commentedDid this never make it back into Drupal 7? I'm trying to eliminate the need for an EU cookie popup on our site and I think the login cookie is that last thing. If I could just get it to say something like 'Remember me with a cookie' I could turn the popup off.
Comment #18
gapple@somnolentsurfer you can implement a
form_alterto adjust the text as needed for your site, but it is currently not possible through the UI.The persistent login cookie is functionally and technically equivalent to the session cookie, so would it actually require any different or additional notification?
Comment #19
gappleClosing out 7.x issues.