Also folgende Sachen sind mir aufgefallen.
Du nutzt kein richtiges Less sondern reines CSS.(Falls du nicht weißt wie das aussehen könnte/sollte frag nach )
Du schließt in HTML den <br> Tag das ist unnötig und nicht gültig. Der Tag muss laut xHTML Standard so aussehen <br /> Das reicht. Damit hast du Ihn direkt geschlossen auch wenn das schließend unnötig ist
Des weiteren sind die if Abfragen im Template unnötig für die Uhrzeit diese könntest du auch in die Sprachvariabel packen und hättest weniger code.
Sähe ggfs. dann so aus.
<category name="wcf.global">
<item name="wcf.global.welcomeBackMessage.goodnight">Gute{if $hour < 23 || $hour > 6}n{/if} {if $hour > 23 || $hour < 6}Nacht{elseif $hour > 5 && $hour < 10}Morgen{elseif $hour > 17}Abend{/if}]]></item>
</category>
Wäre jetzt nur ein Beispiel.
Ausserdem ist die Struktur(Ich denke du verwendest keine IDE) aller Dateien etwas komisch und nicht gut lesbar, man rückt in der Regel Sachen ein sofern diese untergeordnet sind z.b. so
<category name="wcf.acp.group">
<item name="wcf.acp.group.option.category.user.welcomegruss"><![CDATA[Willkommensgruß]]></item>
<item name="wcf.acp.group.option.user.welcomegruss.canViewInfo"><![CDATA[Kann Information sehen]]></item>
</category>
Das ganze gilt aber auch für Template und anderen
ich hoffe ich konnte dir da weiter helfen.