Fixed Missing <p> element in a template

Luxus

Well-known member
This is the lost_password template:

Rich (BB code):
<xen:title>{xen:phrase lost_password}</xen:title>

<form action="{xen:link 'lost-password/lost'}" method="post" class="xenForm formOverlay">
	{xen:phrase if_forgotten_your_password_use_form_to_reset}
	
	<dl class="ctrlUnit">
		<dt><label for="ctrl_username_email">{xen:phrase name_or_email}:</label></dt>
		<dd><input type="text" name="username_email" class="textCtrl" id="ctrl_username_email" autofocus="true" /></dd>
	</dl>

	<xen:if hascontent="true">
		<fieldset>
			<xen:contentcheck>
				<xen:include template="helper_captcha_unit" />
			</xen:contentcheck>
		</fieldset>
	</xen:if>
	
	<dl class="ctrlUnit submitUnit">
		<dt></dt>
		<dd><input type="submit" value="{xen:phrase reset_password}" accesskey="s" class="button primary" /></dd>
	</dl>
	
	<input type="hidden" name="_xfToken" value="{$visitor.csrf_token_page}" />
</form>

The red part is the phrase that's missing a <p> element. Because of this you can't directly style the message without affecting the <form> element. It doesn't seem to be a big deal and I can fix it myself within an inch of a second, but the template will be marked as edited just because I added a <p> element to a paragraph, something that is supposed to be there from the first place. Please add a <p> element to this phrase.
 
Independently, this isn't really a bug -- more of a general choice. However, every other instance where I found this style of usage, we did include a <p> tag so this one is inconsistent.
 
Back
Top Bottom