|
Usage of <label for=""> misses corresponding id
|
|
The way <label> is used makes it rather useless
|
Category Unknown |
Affected Version 1.6 rc5 |
Priority 7 |
Status Confirmed |
Fixed Version (none) |
|
Submitted 01-27-2010 |
|
|
|
Assigned Users
(none)
|
Tags
(none)
|
|
01-27-2010 06:03 PM
|
|
|
|
Usage of <label for=""> misses corresponding id
The way <label> is used makes it rather useless
Affects file include/staff/viewticket.inc.php
Example
PHP Code:
<label for="ticket_status"><b>Ticket Status:</b></label>
...
<input type="checkbox" name="ticket_status" value="Reopen" <?=$checked?> > Reopen on Reply
Here the last line misses an id="something". See HTML specs for <label>.
Fix (for example)
PHP Code:
<label for="cb_ticket_status"><b>Ticket Status:</b></label>
...
<input type="checkbox" name="ticket_status" id="cb_ticket_status" value="Reopen" <?=$checked?> > Reopen on Reply
Or if one wants to be able to click "Reopen on Reply" to switch the checkbox, too
PHP Code:
<label for="cb_ticket_status"><b>Ticket Status:</b>
...
<input type="checkbox" name="ticket_status" id="cb_ticket_status" value="Reopen" <?=$checked?> > Reopen on Reply</label>
Note, not all but most occurrences of <label for in include/staff/viewticket.inc.php are affected.
HTH. |
|
01-28-2010 12:31 PM
|
|
|
|
I realize this is a usability concern, and as someone who does, in fact, need accessible software in order to use the web, I can tell you that it's not something we're going to ignore.
The changes may not make it into the next release simply due to time constraints, but parts of the UI will soon be getting a major overhaul and this is one of the issues we will be addressing - labels for all the form fields, etc.
|
All times are GMT -4. The time now is 06:26 PM.
Subscribe to this issue