From 7bb1b8bc97f8f567b66f4d4f634cf9ded6e7d1b1 Mon Sep 17 00:00:00 2001
From: riking <rikingcoding@gmail.com>
Date: Tue, 9 Sep 2014 11:20:25 -0700
Subject: [PATCH] Handlebars structure tweaks to post.handlebars

 - Combine bind-attrs on the <article> (seriously that made no sense)
 - Hoist a bind-attr class to the view, allowing more fancy styling
 - Hoist a <div> outside a conditional
 - Change href='#' to unvalued hrefs
---
 .../discourse/templates/post.js.handlebars     | 18 ++++++++----------
 .../javascripts/discourse/views/post_view.js   |  1 +
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/app/assets/javascripts/discourse/templates/post.js.handlebars b/app/assets/javascripts/discourse/templates/post.js.handlebars
index cb16bd9fee4..e22822e55b4 100644
--- a/app/assets/javascripts/discourse/templates/post.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/post.js.handlebars
@@ -4,10 +4,10 @@
   {{view 'reply-history' content=replyHistory}}
 </div>
 
-<article class='boxed' {{bind-attr id="postElementId"}} {{bind-attr data-post-id="id"}} data-user-id="{{unbound user_id}}">
+<article class='boxed' {{bind-attr id="postElementId" data-post-id="id" data-user-id="user_id"}}>
   <div class='row'>
     {{#if showUserReplyTab}}
-      <a href='#' {{action toggleReplyHistory this target="view"}} class='reply-to-tab'>
+      <a href {{action toggleReplyHistory this target="view"}} class='reply-to-tab'>
         {{#if loadingReplyHistory}}
           {{i18n loading}}
         {{else}}
@@ -18,15 +18,13 @@
       </a>
     {{/if}}
     <div class='topic-avatar'>
-      {{#unless userDeleted}}
-        <div {{bind-attr class=":contents byTopicCreator:topic-creator"}}>
+      <div class="contents">
+        {{#unless userDeleted}}
           {{poster-avatar action="expandPostUser" post=this classNames="main-avatar"}}
-        </div>
-      {{else}}
-        <div class="contents">
+        {{else}}
           <i class="fa fa-trash-o deleted-user-avatar"></i>
-        </div>
-      {{/unless}}
+        {{/unless}}
+      </div>
       {{#if wiki}}
         <div {{action editPost this}} class="wiki" title="{{i18n post.wiki.about}}"><i class="fa fa-pencil-square-o fa-3x"></i></div>
       {{/if}}
@@ -44,7 +42,7 @@
         {{#if hasHistory}}
           <div class='post-info edits'>
             {{#if can_view_edit_history}}
-              <a href='#' class="{{unbound view.historyHeat}}" {{action showHistory this}} title="{{i18n post.last_edited_on}} {{raw-date updated_at}}">
+              <a href class="{{unbound view.historyHeat}}" {{action showHistory this}} title="{{i18n post.last_edited_on}} {{raw-date updated_at}}">
                 {{editCount}}
                 <i class='fa fa-pencil'></i>
               </a>
diff --git a/app/assets/javascripts/discourse/views/post_view.js b/app/assets/javascripts/discourse/views/post_view.js
index bc7fe08f8af..743a89a0ad1 100644
--- a/app/assets/javascripts/discourse/views/post_view.js
+++ b/app/assets/javascripts/discourse/views/post_view.js
@@ -7,6 +7,7 @@ Discourse.PostView = Discourse.GroupedView.extend(Ember.Evented, {
                       'selected',
                       'post.hidden:post-hidden',
                       'post.deleted',
+                      'byTopicCreator:topic-creator',
                       'groupNameClass'],
   postBinding: 'content',