Closed (fixed)
Project:
Webform
Version:
7.x-3.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Apr 2013 at 01:03 UTC
Updated:
4 Jan 2014 at 03:28 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rooby commentedPatch to add invisible labels, so screen readers can read the label.
Comment #2
rooby commentedNew version that doesn't add a label for the time period because the individual radio button elements have their own labels.
Comment #3
quicksketchThanks this is a good suggestion. Minor problem here:
$none doesn't exist right? This should probably just be NULL, and in that case you probably don't need to set #title_display immediately afterwards.
Comment #4
rooby commentedThe purpose here is to set titles for screen readers so setting them to null would not help.
$none is the visible value of the empty/none option for each of the date parts, which is either t('Day'), t('Month'), or t('Year').
This is the code in question:
Comment #5
quicksketchAh, okay sorry I should have applied the patch to review it. I'll take another look at this today.
Comment #6
quicksketchThanks, committed to D7 versions.