Remove unnecessary inner-tag whitespace and self-closing tags

Self-closing tags are so XHTML. HTML5 allows them on some elements, but
doesn't require them. Cleaning up for consistency and for a few bits
saved to clients.

Change-Id: I781b8d79bbd5dd1ee75a167718964c1142ecfc5d

includes/templates/SearchBox.mustache

@@ -24,8 +24,8 @@
 					type="search" name="search"
 					placeholder="{{msg-searchsuggest-search}}"
 					{{/is-primary}}
-				/>
-				<input type="hidden" name="title" value="{{page-title}}"/>
+				>
+				<input type="hidden" name="title" value="{{page-title}}">
 				{{!	We construct two buttons (for 'go' and 'fulltext' search modes), but only one will be
 						visible and actionable at a time (they are overlaid on top of each other in CSS).
 						* Browsers will use the 'fulltext' one by default (as it's the first in tree-order),
@@ -35,9 +35,9 @@
 							the 'fulltext' button and handles 'fulltext' search itself; this will reveal the 'go'
 							button and cause it to be used.	!}}
 				<input{{#is-primary}} id="mw-searchButton"{{/is-primary}}
-					{{{html-button-fulltext-attributes}}} value="{{msg-searchbutton}}" />
+					{{{html-button-fulltext-attributes}}} value="{{msg-searchbutton}}">
 				<input{{#is-primary}} id="searchButton"{{/is-primary}}
-					{{{html-button-go-attributes}}} value="{{msg-searcharticle}}" />
+					{{{html-button-go-attributes}}} value="{{msg-searcharticle}}">
 			</div>
 		</form>
 	</div>

includes/templates/UserLinks.mustache

@@ -1,4 +1,4 @@
-<nav class="vector-user-links{{#is-wide}} vector-user-links-wide{{/is-wide}}" aria-label="{{msg-personaltools}}" role="navigation" >
+<nav class="vector-user-links{{#is-wide}} vector-user-links-wide{{/is-wide}}" aria-label="{{msg-personaltools}}" role="navigation">
 	{{#data-user-more}}{{>Menu}}{{/data-user-more}}
 	{{#data-user-menu}}{{>Menu}}{{/data-user-menu}}
 </nav>