search

hoscanoa / struts2-bootstrap

Automatically exported from code.google.com/p/struts2-bootstrap
0 stars 0 forks source link

labels for textfields do not have appropriate for= attribute #6

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?

    <@s.textfield theme="bootstrap" spellcheck="false" id='loginUsername' name="loginUsername" label="Username" cssClass="required" />

produces:

<div class="control-group">
    <label class="control-label">Username</label>
    <div class="controls">
    <input type="text" name="loginUsername" value="" id="loginUsername" class="required valid" spellcheck="false">    </div> 
</div>

instead of:

<div class="control-group">
    <label for="loginUsername" class="control-label">Username</label>
    <div class="controls">   
      <input type="text" name="loginUsername" value="" id="loginUsername" class="required valid" spellcheck="false">    </div> 
</div>

Original issue reported on code.google.com by abrin...@gmail.com on 30 Oct 2012 at 4:38

GoogleCodeExporter commented 9 years ago 
Which version do you use?

<s:textfield name="topic" value="%{item.topic}" label="TODO" tooltip="TODO 
Topic" required="true" theme="bootstrap"/>

Generates in my app:

<div class="control-group "><label class="control-label" 
for="todoEditForm_topic">TODO <span class="required">*</span>   <i 
data-original-title="TODO Topic" class="icon-info-sign s2b_tooltip"></i>
</label><div class="controls">

<input name="topic" value="" id="todoEditForm_topic" type="text"></div>
</div>

Original comment by johgep on 30 Oct 2012 at 6:28

GoogleCodeExporter commented 9 years ago 
We're using the latest available via maven (1.5.1)

Original comment by abrin...@gmail.com on 30 Oct 2012 at 7:29