Not signed in (Sign In)
 
Sep 12th 2007 edited
 
sjeeps just noticed that there's an extra space in the theme/head.php body tag:

Context->BodyAttributes.'>



the use of $Context->BodyAttributes in the core include the space so we could remove it:

line 83 of People.Control.SignInForm.php:$this->Context->BodyAttributes = " onload=\"Focus('txtUsername');\"";line 43 of Vanilla.Control.SearchForm.php:$this->Context->BodyAttributes .= " onload=\"Focus('txtKeywords');\"";

Usually the space is not included in the template; you have to include it when set the calue of the attribute property. comment_form.php:<textarea name="Body" class="'
.($this->Context->Session->User->Preference('ShowLargeCommentBox') ? 'LargeCommentBox' : 'SmallCommentBox')
.'" id="CommentBox" rows="10" cols="85"'.$this->CommentFormAttributes.'>'
.$Comment->Body
.'</textarea>


Still, fixing it could break an extension with no real benefits; it might be better to leave it for vanilla 1.1.3 and fix it 1.2.

Issue information

  • 17
  • No assigned user

    No assigned user

     

  • On Hold
  • Low
  • Bug

Vanilla 1.1.5 is a product of Lussumo. More Information: Documentation, Community Support.