Radio buttons in email selector need labels

Bug #1277297 reported by Jono M
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Jono M

Bug Description

For people using screen readers, having a label for every form input is essential as they have no other (easy) way of knowing what the input is for. In the email list pieforms element (used on the Profile page) emails are displayed as radio buttons with labels, but the labels are not linked to the actual buttons.
The labels should probably use the same method as normal radio button elements and include both the element title and the individual email.

Affects latest master

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/2991

Jono M (jonom)
Changed in mahara:
status: New → In Progress
assignee: nobody → Jono Mingard (mingard)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/2991
Committed: http://gitorious.org/mahara/mahara/commit/73d43a1f1c48a12afdc5ed9ba11d8908712d6c59
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 73d43a1f1c48a12afdc5ed9ba11d8908712d6c59
Author: Jono Mingard <email address hidden>
Date: Fri Feb 7 13:54:08 2014 +1300

Use labels in email address element (Bug #1277297)

Linked email labels to radio buttons (and added element title as hidden
text). Also added the 'btn' class to the add email button

Change-Id: Id7a736b33abbcb18c8682f9a6e8e7a469424b16e
Signed-off-by: Jono Mingard <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.