The dialog asks a yes/no question, but the buttons in the dialog are "Cancel" and "OK". "Cancel" is not a valid response to "Do you want to reset?" Neither is "OK", literally. So, this is a bad default for the logout message.

Suggest changing to something similar to the the following:
"Your session is about to expire. Click OK to continue remain logged in."

(Note that I'm not sure it'd be OK to say "Click" here since a user need not click on a button to press it. Maybe "Press" is more proper.)

CommentFileSizeAuthor
#4 Autologout Alert.png4.3 KBroball

Comments

threexk’s picture

threexk commented

s/continue remain logged in/remain logged in/ in previous comment.

levelos’s picture

levelos commented

The dialog message is completely configurable, are you suggesting a better default? It's also relevant to remember that those who are using jquery_ui will have a very different experience since they are not limited to native browser confirm dialog and it's button choices.

threexk’s picture

threexk commented

Yes, I'm suggesting a default that makes sense with the buttons in the browser's confirm dialog ("OK" and "Cancel"). I know it is configurable, but I think it is important to have sensible defaults.

I don't know what the buttons are in the default dialog if you are using jQuery UI.

roball’s picture

roball commented
StatusFileSize
new Autologout Alert.png4.3 KB

The buttons on my Autologout Alert screen are (per default) labelled with [ Reset ] and [ Logout ] (see attached screenshot).

izmeez’s picture

izmeez commented

I don't have an issue with the default message and names of buttons.

Rest moved to new issue,

However, I have noticed that anytime the autologout module configuration is changed, even just checking or unchecking some boxes the text of the message to display gets extra <p> and </p> added even to the default text. I haven't a clue why this happens but it makes the message displayed messy.

<p>&lt;p&gt;Your session is about to expire. Do you want to reset it?&lt;/p&gt;</p>

Gosh, I guess I should open this as a new issue :-(

johnennew’s picture

johnennew commented
Status: Active » Closed (fixed)

I am marking this as fixed as I believe it now is in the 6.x-4.x and 7.x-4.x branches. The text on the buttons is translatable.

Feel free to reopen with more information if it is still a problem.