View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
05846 | Bug reports | Survey taking | public | 2012-02-22 19:47 | 2012-03-14 21:08 |
Reporter | DenisChenu | Assigned To | DenisChenu | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 1.92RC4 | ||||
Target Version | 1.92 | Fixed in Version | 1.92 | ||
Summary | 05846: Label on question texte can be a bad solution | ||||
Description | We put some label for the question text. But it can be a proble: There are sur other problem. ( link for example ...) | ||||
Steps To Reproduce | Add some link , div presentation on a question type text. | ||||
Additional Information | There are a exemple here: I don't see a good way ti have somle labeling for text question, maybe:
( i don't know where to find LS buil number with git, but it's the last | ||||
Tags | No tags attached. | ||||
Bug heat | 10 | ||||
Complete LimeSurvey version number (& build) | 00000 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | not relevant | ||||
Database type & version | Mysql 5.1.49 | ||||
Server OS (if known) | debian/linux | ||||
Webserver software & version (if known) | apache | ||||
PHP Version | PHP Version 5.3.3-7 | ||||
child of | 04948 | closed | "label for' tags for certain forms fields |
DenisChenu, |
So, <label> should use the for='id' attribute for associating labels with elements, rather than trying to surround the elements. |
|
<label for="thename">Some information</label><input name="thename" /> id is a very good optimisation: But label mee "Some information", <label for="thename"><input name="thename" /></label> is NOT a solution. I don't have THE solution, i think:
|
|
Given that WAI-ARIA just got W3C candidate recommendation status last month, I think it is a bad idea to try to support it right now. Also, we'd need to continue to use <label> to provide backwards support for older browsers. WAI-ARIA is good to think about for optional HTML 5 support in the future. |
|
Do we have anyone who can provide feedback from a Section 508 compliance perspective (that's the USA set of rules), or comparable international ones? I'm not well versed in that area. Currently, <label> is nice because you can click on the text and toggle the value of the associated control. So, if you have a checkbox or radio button, you can click on the text rather than exactly hitting the input control. However, <label> does nothing for sighted users for array-style questions. So, do we really need it there? Also, for <textarea> and <input type='text'>, clicking on the text does shift focus to the input box, but it doesn't toggle the value. Is such a shift of focus an accessibility goal? If not, then I see no harm in removing those <label> tags. |
|
All input need a label for accessibility. Label mean : "What is the input for ?" You can try http://www.cynthiasays.com/ , but you can use too http://lynx.isc.org/ . Best is to have some information from user. I have a link to answer of a survey by a concurrent system: no javascript => no answer . I think LimeSurvey have to be accessibility best compatible. Denis |
|
Reminder sent to: Hello magiclko, You make the modification for "label for' tags for certain forms fields. I think the modification can be good for some survey, but can be very bad for other. Question text can be very long, can have some link, some "javascript link" ( show image for example). I think easiest way are : With hide, you remove from visibility for all user, but show for blind user. After, we can add : aria-required, aria-describedby. Denis |
|
Hello, I ask at a accessibility mailing list, and the answer is : I think too of
And maybe some optimisation for accessibility without touching of visibility without screen reader. Denis PS: had to work on table too ... Arf .... table accessibility first. |
|
@TMSWhite: That's exactly the reason why I requested in 04948 that this would be an optional feature which can be switched on/off in the question attributes (and defaults to true).
|
|
After reading Shnoulles comment again I am tending to support the suggestion to use the invisible label instead (if that's really visible to the screen reader). |
|
Hello: positon:absolute,top=-9999px,left:-9999px : reading ( actual hide css in all included template, maybe some correction ( left:auto ) but it's a solution. I don't know for opacity:0, but i think reading. But there another good alternative i think:
We can use this optionnal attribute at some question to have some specific accessibility issue on another question ( fieldset on radio list for example).. for hide in user template : if i had time for http://bugs.limesurvey.org/view.php?id=5807 for 2.0, it can be very good. But included template have this class for a long time ago. |
|
1.92 released |
|
LimeSurvey: Yii c83cc198 2012-02-28 12:37:20 Details Diff |
Fixed issue 05846 and 04948: accessibility for some question type : add label Dev : Add label for short-text long-text huget-text numeric date list-dropdown langage switch |
Affected Issues 05846 |
|
mod - application/helpers/qanda_helper.php | Diff File | ||
mod - application/helpers/replacements_helper.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-02-22 19:47 | DenisChenu | New Issue | |
2012-02-22 20:15 | TMSWhite | Note Added: 17565 | |
2012-02-23 08:48 | DenisChenu | Note Added: 17576 | |
2012-02-23 15:04 | TMSWhite | Note Added: 17580 | |
2012-02-23 16:08 | TMSWhite | Note Added: 17586 | |
2012-02-23 18:25 | DenisChenu | Note Added: 17597 | |
2012-02-23 18:34 | DenisChenu | Note Edited: 17597 | |
2012-02-24 09:40 | DenisChenu | Issue Monitored: DenisChenu | |
2012-02-24 09:41 | DenisChenu | Relationship added | child of 04948 |
2012-02-24 09:58 | DenisChenu | Issue Monitored: magiclko | |
2012-02-24 09:58 | DenisChenu | Note Added: 17602 | |
2012-02-24 10:05 | DenisChenu | Note Added: 17603 | |
2012-02-24 10:06 | DenisChenu | Note Edited: 17603 | |
2012-02-26 14:13 | c_schmitz | Assigned To | => TMSWhite |
2012-02-26 14:13 | c_schmitz | Status | new => assigned |
2012-02-26 14:13 | c_schmitz | Assigned To | TMSWhite => magiclko |
2012-02-26 14:19 | c_schmitz | Note Added: 17639 | |
2012-02-27 00:21 | c_schmitz | Note Added: 17659 | |
2012-02-27 09:29 | DenisChenu | Note Added: 17662 | |
2012-02-27 09:29 | DenisChenu | Note Edited: 17662 | |
2012-02-28 20:51 | DenisChenu | Assigned To | magiclko => DenisChenu |
2012-02-29 18:38 | DenisChenu | Status | assigned => resolved |
2012-02-29 18:38 | DenisChenu | Fixed in Version | => 1.92 |
2012-02-29 18:38 | DenisChenu | Resolution | open => fixed |
2012-03-01 16:06 | TMSWhite | Target Version | => 1.92 |
2012-03-03 02:25 | c_schmitz | Note Added: 17726 | |
2012-03-03 02:25 | c_schmitz | Status | resolved => closed |
2012-03-14 21:08 | DenisChenu | Changeset attached | => Import 2012-03-09 13:30:34 Yii c83cc198 |