Use handlebars helpers wherever possible

app/assets/stylesheets/stream_element.scss

@@ -53,12 +53,12 @@
         width: 50px;
       }
     }
-    .author {
+    .author-name {
       font-weight: bold;
       margin-bottom: 4px;
       unicode-bidi: bidi-override;
     }
-    a.author { color: $blue; }
+    a.author-name { color: $blue; }
     .feedback {
       margin-top: 5px;
       font-size: 11px;

app/assets/templates/comment_tpl.jst.hbs

@@ -20,16 +20,14 @@
     {{/if}}
     </div>
 
-    {{#with author}}
+    {{#linkToAuthor author}}
-      <a href="/people/{{guid}}" class="author author-name {{hovercardable this}}">
+      {{name}}
-        {{name}}
+    {{/linkToAuthor}}
-      </a>
-    {{/with}}
 
     <div class="collapsible comment-content">
       {{{text}}}
     </div>
-    
+
     <div class="info">
       <a href="/posts/{{parent.id}}#{{guid}}" class="permalink_comment">
         <time class="timeago" data-original-title="{{{localTime created_at}}}" datetime="{{created_at}}"/>

app/assets/templates/header_tpl.jst.hbs

@@ -72,7 +72,7 @@
       <div class="user-menu-more-indicator">
         ▼
       </div>
-      <img alt="{{current_user.name}}" class="avatar user-menu-avatar" src="{{current_user.avatar.small}}" title="{{current_user.name}}" />
+      {{{personImage current_user 'small' 'avatar user-menu-avatar'}}}
       <a class="user-name" href="#">{{current_user.name}}</a>
     </li>
     <li class="user-menu-item"><a href="/people/{{current_user.guid}}">{{t "header.profile"}}</a></li>

app/assets/templates/likes-info_tpl.jst.hbs

@@ -12,11 +12,9 @@
         {{else}}
 
           {{#each likes}}
-            {{#with author}}
+            {{#linkToAuthor author}}
-              <a href="/people/{{guid}}" class="{{hovercardable this}}">
+              {{{personImage this 'small' 'micro'}}}
-                <img src="{{avatar.small}}" class="avatar micro" title="{{name}}"/>
+            {{/linkToAuthor}}
-              </a>
-            {{/with}}
           {{/each}}
         {{/unless}}
       </div>

app/assets/templates/reshare_tpl.jst.hbs

@@ -6,14 +6,16 @@
 
       {{#with root}}
         <a href="/people/{{author.guid}}" class="img {{{hovercardable this}}}">
-          <img src="{{author.avatar.small}}" class="avatar" />
+          {{{personImage author 'small'}}}
         </a>
       {{/with}}
 
       {{#with root}}
         <div class="bd">
           <div>
-            <a href="/people/{{author.guid}}" class="author {{{hovercardable author}}}">{{author.name}}</a>
+            {{#linkToAuthor author}}
+              {{name}}
+            {{/linkToAuthor}}
 
             <span class="details grey">
               -

app/assets/templates/stream-element_tpl.jst.hbs

@@ -37,9 +37,9 @@
     {{/if}}
 
     <div>
-      {{#with author}}
+      {{#linkToAuthor author}}
-        <a href="/people/{{guid}}" class="author {{{hovercardable this}}}">{{name}}</a>
+        {{name}}
-      {{/with}}
+      {{/linkToAuthor}}
 
       <span class="details grey">
         -

features/step_definitions/hovercard_steps.rb

@@ -16,6 +16,6 @@ end
 
 When (/^I hover "([^"]*)" within "([^"]*)"$/) do |name, selector|
   with_scope(selector) do
-    find(".author", text: name).hover
+    find(".author-name", text: name).hover
   end
 end