| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 0.6.2 |
| Bug #8523 | "date" element label not showing up | ||
|---|---|---|---|
| Submitted: | 2006-08-21 04:29 UTC | ||
| From: | eduardo593734256 at gmail dot com | Assigned: | wiesemann |
| Status: | Closed | Package: | HTML_QuickForm_Renderer_Tableless (version 0.3.3) |
| PHP Version: | Irrelevant | OS: | Windows |
| Roadmaps: | (Not assigned) | ||
[2006-08-21 04:29 UTC] eduardo593734256 at gmail dot com
(Eduardo Marinho)
Description:
------------
The select id is not being generated.
Test script:
---------------
$form->addElement('date', 'inicio', 'Início:', array('language' => 'pt-br', 'format' => 'H:i', 'optionIncrement' => array('i' => 5)));
Expected result:
----------------
Actual result:
--------------
:
:
Comments
[2006-08-23 14:55 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2006-08-24 06:19 UTC] eduardo593734256 at gmail dot com
Hi Mark,
The invalid id causes validation problems (XHTML 1.1 @ validator.w3.org).
My first idea was to give consistency to this kind of label: clicking on it would put keyboard focus on the select element. Now I noticed that this doesn't happen in any popular web browser.
Removing the "for" attribute would require the label to wrap the select element, which cannot wrap <div class="qfelement"> because that would mean a new lot of validation errors. My guess is that it would require some css and/or <div> rework to validate the page.
Thank you for your attention.
--
Eduardo Marinho
[2006-08-24 11:49 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2006-08-24 16:19 UTC] eduardo593734256 at gmail dot com
Hi Mark,
I don't think it's YOUR bad english. :-)
I'll say it in HTML:
Citing :
for = idref [CS]
This attribute explicitly associates the label being defined with another control. When present, the value of this attribute must be the same as the value of the id attribute of some other control in the same document. When absent, the label being defined is associated with the element's contents.
Since we can't call the whole group a single element and give it a "for", the only other solution
would be to add a "for" pointing to the first select element.
But validation also complains about "[" in "inicio[H]" in the id, so you could change it to just "inicio", for example:
Thank you for reading.
--
Eduardo Marinho
:
:
:
:
[2006-08-24 17:00 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2006-09-12 12:22 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!