View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|11784||Feature requests||Survey taking||public||2016-10-12 08:15||2018-04-06 11:48|
|Target Version||3.0||Fixed in Version||3.0|
|Summary||11784: Replace alert + confirm in core system (?)|
|Description||Currently : |
- default template replace alert by boostrap dialog
- default template don't replace confirm (due to langage string lack)
Maybe we must use some replacement in core.
|Additional Information||Pros :|
- Seems better on some screen
- Can update look according to template
- In some, screen : the real alert/confirm is more readable (little phone for example).
|Tags||No tags attached.|
In fact : it's not a feature request : really don't know what core must do.
Maybe best choice is :
- use alert and confirm function
- let template manager update this js function if he want (do it in default template)
=> Then we can just add a array of some language string : needed : Yes/No/Close/(Cancel ?)/(Confirm ?)
Currently I would vote for "let template manager update this js function".
I still don't know why alert system is default.
These popups save reload time, but since browser can offer to suppress alerts I never go for them. Are there big fans of the alert system? I can remember a time where showpopups=0 was the default setting in LS.
In LS 2.5 the showpopups=0 scope won't prevent dialogs in JS in certain situations, right? Cause I never saw a yes/no/cancel/confirm dialog inside html.
A info/alert box above the first question with html dialog would be my favor.
confirm is used for Clear all button only actually.
alert can be easily replaced in template : default template do it, but we can easily replace by a div at start of survey.
alert is used when come back to a survey : if you don't answer a mandatory question for example.
Maybe add some "langage string tool" :)
Fixed in https://bugs.limesurvey.org/view.php?id=11970
core package use alert/confirm in a new function. template-default package update this function, template.js can update this function.
|2016-10-12 08:15||DenisChenu||New Issue|
|2016-10-12 08:17||DenisChenu||Note Added: 41244|
|2016-10-12 11:36||jelo||Note Added: 41250|
|2016-10-12 11:40||DenisChenu||Note Added: 41252|
||Product Version||=> 3.0|
||Target Version||=> 3.0|
|2016-12-10 14:22||DenisChenu||Relationship added||related to 11977|
|2016-12-10 14:22||DenisChenu||Relationship added||related to 11970|
|2016-12-10 14:24||DenisChenu||Note Added: 42512|
|2016-12-10 14:24||DenisChenu||Assigned To||=> DenisChenu|
|2016-12-10 14:24||DenisChenu||Status||new => resolved|
|2016-12-10 14:24||DenisChenu||Resolution||open => fixed|
|2016-12-10 14:24||DenisChenu||Fixed in Version||=> 3.0|
||Status||resolved => closed|