diff options
author | Joey Hess <joey@kitenet.net> | 2010-06-15 13:33:43 -0400 |
---|---|---|
committer | Joey Hess <joey@kitenet.net> | 2010-06-15 13:33:43 -0400 |
commit | 919df4ae9c0372506feebee41821e4de2c78cb91 (patch) | |
tree | 7393dc4bb74dcd2cad257cd77153984a25050708 | |
parent | 4e18d4415b49bbcc4a43db22ee4473a8c99f0eb2 (diff) |
fix labels for form elements
The label for attribute must correspond to the element id (not name).
And it needs to be unique inside the loop.
-rw-r--r-- | templates/commentmoderation.tmpl | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/templates/commentmoderation.tmpl b/templates/commentmoderation.tmpl index 10f2b36df..dfa63c20b 100644 --- a/templates/commentmoderation.tmpl +++ b/templates/commentmoderation.tmpl @@ -4,7 +4,7 @@ <input type="hidden" name="do" value="commentmoderation" /> <input type="hidden" name="sid" value="<TMPL_VAR SID>" /> <input type="submit" value="Submit" /> -<input type="checkbox" name="rejectalldefer" value="1" /> +<input id="rejectalldefer" type="checkbox" name="rejectalldefer" value="1" /> <label for="rejectalldefer">Reject all comments marked <em>Defer</em></label> <br /> <TMPL_LOOP COMMENTS> @@ -12,18 +12,18 @@ <div> <TMPL_VAR VIEW> </div> -<input type="radio" value="Defer" name="<TMPL_VAR ID>" checked="checked" /> -<label for="Defer">Defer</label> -<input type="radio" value="Accept" name="<TMPL_VAR ID>" /> -<label for="Accept">Accept</label> -<input type="radio" value="Reject" name="<TMPL_VAR ID>" /> -<label for="Reject">Reject</label> +<input id="defer_<TMPL_VAR ID>" type="radio" value="Defer" name="<TMPL_VAR ID>" checked="checked" /> +<label for="defer_<TMPL_VAR ID>">Defer</label> +<input id="dccept_<TMPL_VAR ID>" type="radio" value="Accept" name="<TMPL_VAR ID>" /> +<label for="dccept_<TMPL_VAR ID>">Accept</label> +<input id="reject_<TMPL_VAR ID>" type="radio" value="Reject" name="<TMPL_VAR ID>" /> +<label for="reject_<TMPL_VAR ID>">Reject</label> </div> <br /> </TMPL_LOOP> <input type="submit" value="Submit" /> -<input type="checkbox" name="rejectalldefer" value="1" /> -<label for="rejectalldefer">Reject all comments marked <em>Defer</em></label> +<input id="rejectalldefer2" type="checkbox" name="rejectalldefer" value="1" /> +<label for="rejectalldefer2">Reject all comments marked <em>Defer</em></label> </form> <TMPL_ELSE> <p> |