Originally posted by Matt Zimmermann
View Post
Announcement
Collapse
No announcement yet.
Ready Theme errors. Customer Fields script OCST, ACAD, ACED
Collapse
X
-
- 1 like
-
My 2 Cents.
Either way , l- or js- ... I understand the idea of being consistent by adding the prefix js- in front of id's to indicate that this element is being accessed by JavaScript and not just a style.
However, with regard to form fields, I believe it to be more consistent to make the id and the name the same and avoid having the <label for=""> attribute
being incorrect as in the example above. This would also allow scripts written for one site to be used as is on another site.
I'm re-coding my javascript to use form element names.
Leave a comment:
-
Hi Bruce,
I see the confusion. Although I typed a lowercase "L" and made it italic, it looks like the number "1". You are correct in your thinking though.
Leave a comment:
-
hmmm....i thought starting element IDs with a number was at least, poor stylistically and can effect some jquery CSS functions such as querySelector?
Leave a comment:
-
Hi Ray,
From a web development standpoint, we only use IDs, in older themes, to trigger JavaScript, hence most of our IDs are prefixed with js- . When an ID is being used in conjunction with a LABEL, the the ID is prefixed with l-, unless there is JavaScript involved. Unfortunately, when it comes to SELECT elements being delivered by the software, there may or may not be an actual ID attached to them. At that point, only the NAME attribute can be used to trigger scripts or the ID has to be added dynamically as we do withe the Booc ReadyTheme.
Leave a comment:
-
Ready Theme errors. Customer Fields script OCST, ACAD, ACED
I'm currently working with Iron and Wool, but this affects other themes as well.
In the template for Customer Fields, the id attribute for the input fields has been changed so that the field name attribute is no longer the same as the id attribute.
While mostly just annoying, there are some cases where it fails. For example:
Code:<label class="required" for="l-Customer_ShipStateSelect">State/Province:</label> <mvt:item name="states" param="Customer_ShipStateSelect" /> <!-- output id attribute does not match --> <label class="required" for="l-Customer_ShipCountry">Country:</label> <mvt:item name="countries" param="Customer_ShipCountry" /> <!-- output id attribute does not match -->
Just for my information, what is reason for the field id and the field name to no longer match?Last edited by RayYates; 07-05-16, 08:38 AM.Tags: None
Leave a comment: