LimeSurvey issue tracker
Registration

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
07967Bug reports[All Projects] Accessibilitypublic2013-07-03 09:172013-10-11 11:08
Reporternicola_86 
Assigned ToDenisChenu 
PrioritynormalSeverityminor 
StatusclosedResolutionfixed 
Product Version2.00+ 
Target Version2.05 RCFixed in Version2.05 RC 
Summary07967: Without javascript there are still present button that don't work in the survey : back, clear the survey, send it incomplete.
DescriptionWhile using the survey without javascript in the browser, the user cannot use the buttons that are on the interface: back, clear the survey and the one to send it incomplete. Another issue is the progress bar on the top, without the progress bar it's not clear why is present: "0% 100%"
Steps To Reproducedisable javascript, put the survey in a way to show one question for page and excute the survey
I will donate to the project if issue is resolved within 48 hrsNo
LimeSurvey build number OR git commit ID130428
BrowserMozilla Firefox
Database & DB-VersionOracle
Operating System (Server)CentOS
Webserver software & versionApache/2.2.15
PHP VersionPHP/5.3.3,
Attached Files

- Relationships
related to 07991closedsammousa No js file included in welcome page 
parent of 08121closedDenisChenu Missing classes for "resume later" button cause template elements to look different 
parent of 08186closedDenisChenu Can not use "language changer" without javascript 

-  Notes
User avatar (25704)
Mazi (developer)
2013-07-04 09:53

I think there should be a warning message if JavaScript is turned off, because you can't run a survey without activated JS at all.
User avatar (25707)
DenisChenu (developer)
2013-07-04 10:07

We can run a survey without javascript .....
 And WE have too ...
User avatar (25708)
Mazi (developer)
2013-07-04 10:23

How can that work? There is JS used for submitting the buttons and what about conditions? JS is used for on page validation.
User avatar (25709)
DenisChenu (developer)
2013-07-04 10:28
edited on: 2013-07-04 10:52

Default behaviour : move = movenext.

For condition : js + PHP.

Try it : it work. Best is to use :
- First question
- Second question, mandatory but hidden by condition on the first question

Have other bug/feature request for condition :
http://bugs.limesurvey.org/view.php?id=7959 [^]

User avatar (25770)
DenisChenu (developer)
2013-07-12 10:45

Question:

IE6 don't manage good button. It can be reviewed in javascript . But for PHP: what to do ?
If $move!="" and !(in_array($move,$aAcceptedMove)){
$move='movenext'/'movesubmit";
}

Hard to find the best way ...

Think it's better to have accessibility for all button without javascript except for IE6. But IE6 need default value here. Don't want to add an input hidden just for IE6 ....
User avatar (25771)
DenisChenu (developer)
2013-07-12 10:46

Add the relationship because can not do test with IE6 with no js in welcome page.
User avatar (25780)
DenisChenu (developer)
2013-07-12 19:57

Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12737 [^]
User avatar (25781)
DenisChenu (developer)
2013-07-12 19:57

Partial : need save/load
Partial put some js for IE6, but no tested
User avatar (25782)
DenisChenu (developer)
2013-07-13 02:20

Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12739 [^]
User avatar (25951)
DenisChenu (developer)
2013-08-10 11:26

Problem for save .... Need rewrite ....

Put some comment here.

- Actual 2.05 system work good on all browser, this can be added for Save button except for one think.

If we have only one input:text question, if user click on "ENTER" without javbascript : it's take the first button (or first input submit).

IN navigator, i add a button move value="next" at start of navogator. This fix problem, a user "ENTER" think he go next.
But Save (or clear all) can be before navigator(prev/next) then default behaviour are "Save".


An idea can be:
- Adding a display:none button at start of navigator.pstpl (think replacement helper can knwo if we are at navigator.pstpl).
- This button can have value="default": default behaviour can be : movenext OR dontmove and send alert (maybe in plugin for 2.06).

It's OK like that ?
Assign it to me after reading and advice.
User avatar (26232)
c_schmitz (administrator)
2013-09-15 18:50

We do not support IE6 anymore so there is no special IE6 workaround needed.

I really don't understand why they did not find a solution for this in HTML5.
I think the invisible navigator button is the best option although I still hate it ;).
User avatar (26249)
DenisChenu (developer)
2013-09-16 08:40

Yes,

After reflexion, think best is to have:
I replacement:
call navigator.pstp do this:
<button name="move" value="default" style="display:none;"></button>
And after put the content of navigator.pstpl.
Then we are sur we have allways value default for move result.

Like i say : the only problem is for "only one text question" and send ENTER.
Some of our user use LS like "Input Assistant"
User avatar (26312)
tpartner (developer)
2013-09-22 16:33

Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13037 [^]
User avatar (26313)
tpartner (developer)
2013-09-22 16:36

Just fixed a couple of broken paths Denis :)
User avatar (26336)
DenisChenu (developer)
2013-09-24 12:52

Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13048 [^]
User avatar (26337)
DenisChenu (developer)
2013-09-24 13:00

Please nicola_86 : try with the last 2.05 version (or wait for next 2.05 beta release to test).
User avatar (26709)
c_schmitz (administrator)
2013-10-11 11:08

2.05RC2 released

- Related Changesets
LimeSurvey: 2.05 cd16db1d
Timestamp: 2013-07-12 17:52:52
Author: DenisChenu
Committer: Denis Chenu
Details ] Diff ]
Fixed issue 07967: Accessibility of button without javascript : previous and clear
Dev: todo : save / load
Dev: Fixed issue : double inclusion of js and css
Dev: Fixed a lot of js in qanda after 4a2d74dd2efc0f0af63e01bc59d5706ee465d6c4
mod - application/controllers/survey/index.php Diff ] File ]
mod - application/helpers/frontend_helper.php Diff ] File ]
mod - application/helpers/qanda_helper.php Diff ] File ]
mod - application/helpers/replacements_helper.php Diff ] File ]
add - scripts/array-totalsum.js Diff ] File ]
add - scripts/navigator-countdown.js Diff ] File ]
mod - scripts/survey_runtime.js Diff ] File ]
LimeSurvey: 2.05 3b59b81e
Timestamp: 2013-07-13 00:10:17
Author: DenisChenu
Committer: Denis Chenu
Details ] Diff ]
Fixed issue 07967: Accessibility of button without javascript : index
Dev: Ols javascript of index don't work : move it in survey_runtime function
mod - application/helpers/SurveyRuntimeHelper.php Diff ] File ]
mod - application/helpers/common_helper.php Diff ] File ]
mod - application/helpers/frontend_helper.php Diff ] File ]
mod - scripts/navigator-countdown.js Diff ] File ]
mod - scripts/survey_runtime.js Diff ] File ]
LimeSurvey: 2.05 79f112c3
Timestamp: 2013-09-22 14:33:44
Author: tpartner
Committer: tpartner
Details ] Diff ]
Fixed issue 07967: Without javascript there are still present button that don't work in the survey : back, clear the survey, send it incomplete.
Dev: Fixed a few paths to /styles-public/
mod - application/helpers/qanda_helper.php Diff ] File ]
LimeSurvey: 2.05 151d03b4
Timestamp: 2013-09-24 10:52:23
Author: DenisChenu
Committer: Denis Chenu
Details ] Diff ]
Fixed issue 07967: Without javascript there are still present button that don't work in the survey
Dev: except for change language and index : all work without JS in IE7
Dev: except for change language : all work without JS on modern browser
mod - application/controllers/survey/index.php Diff ] File ]
mod - application/helpers/SurveyRuntimeHelper.php Diff ] File ]
mod - application/helpers/frontend_helper.php Diff ] File ]
mod - application/helpers/replacements_helper.php Diff ] File ]
mod - scripts/survey_runtime.js Diff ] File ]

- Issue History
Date Modified Username Field Change
2013-07-03 09:17 nicola_86 New Issue
2013-07-04 09:52 Mazi Assigned To => DenisChenu
2013-07-04 09:52 Mazi Status new => assigned
2013-07-04 09:53 Mazi Note Added: 25704
2013-07-04 10:07 DenisChenu Note Added: 25707
2013-07-04 10:08 DenisChenu Target Version => 2.05 RC
2013-07-04 10:23 Mazi Note Added: 25708
2013-07-04 10:28 DenisChenu Note Added: 25709
2013-07-04 10:52 DenisChenu Note Edited: 25709 View Revisions
2013-07-12 10:45 DenisChenu Note Added: 25770
2013-07-12 10:46 DenisChenu Relationship added related to 07991
2013-07-12 10:46 DenisChenu Note Added: 25771
2013-07-12 10:46 DenisChenu Priority urgent => normal
2013-07-12 19:57 DenisChenu Changeset attached => LimeSurvey 2.05 cd16db1d
2013-07-12 19:57 DenisChenu Note Added: 25780
2013-07-12 19:57 DenisChenu Resolution open => fixed
2013-07-12 19:57 DenisChenu Note Added: 25781
2013-07-13 02:20 DenisChenu Changeset attached => LimeSurvey 2.05 3b59b81e
2013-07-13 02:20 DenisChenu Note Added: 25782
2013-08-10 11:26 DenisChenu Note Added: 25951
2013-08-10 11:27 DenisChenu Assigned To DenisChenu => c_schmitz
2013-08-10 11:27 DenisChenu Status assigned => feedback
2013-09-02 13:56 DenisChenu Relationship added parent of 08121
2013-09-15 18:50 c_schmitz Note Added: 26232
2013-09-15 18:50 c_schmitz Assigned To c_schmitz => DenisChenu
2013-09-15 18:50 c_schmitz Status feedback => assigned
2013-09-16 08:40 DenisChenu Note Added: 26249
2013-09-22 16:33 tpartner Changeset attached => LimeSurvey 2.05 79f112c3
2013-09-22 16:33 tpartner Note Added: 26312
2013-09-22 16:33 tpartner Assigned To DenisChenu => tpartner
2013-09-22 16:36 tpartner Assigned To tpartner => DenisChenu
2013-09-22 16:36 tpartner Note Added: 26313
2013-09-24 12:52 DenisChenu Changeset attached => LimeSurvey 2.05 151d03b4
2013-09-24 12:52 DenisChenu Note Added: 26336
2013-09-24 12:59 DenisChenu Relationship added parent of 08186
2013-09-24 13:00 DenisChenu Note Added: 26337
2013-09-24 13:00 DenisChenu Status assigned => resolved
2013-09-24 13:00 DenisChenu Fixed in Version => 2.05 RC
2013-10-11 11:08 c_schmitz Note Added: 26709
2013-10-11 11:08 c_schmitz Status resolved => closed


Copyright © 2000 - 2014 MantisBT Team
Powered by Mantis Bugtracker