From 1bfbce7b4542739a3601e0722a975b450e86bfb2 Mon Sep 17 00:00:00 2001
From: Eugen Rochko <eugen@zeonfederated.com>
Date: Fri, 6 Jan 2017 20:24:51 +0100
Subject: [PATCH] Clean up h-card mess of divs

---
 app/helpers/stream_entries_helper.rb       |  8 +++---
 app/views/stream_entries/_status.html.haml | 30 ++++++++++------------
 2 files changed, 18 insertions(+), 20 deletions(-)

diff --git a/app/helpers/stream_entries_helper.rb b/app/helpers/stream_entries_helper.rb
index ae2f575b5..15601a079 100644
--- a/app/helpers/stream_entries_helper.rb
+++ b/app/helpers/stream_entries_helper.rb
@@ -15,10 +15,10 @@ module StreamEntriesHelper
 
   def entry_classes(status, is_predecessor, is_successor, include_threads)
     classes = ['entry']
-    classes << 'entry-reblog' if status.reblog?
-    classes << 'entry-predecessor' if is_predecessor
-    classes << 'entry-successor' if is_successor
-    classes << 'entry-center' if include_threads
+    classes << 'entry-reblog u-repost-of h-cite' if status.reblog?
+    classes << 'entry-predecessor u-in-reply-to h-cite' if is_predecessor
+    classes << 'entry-successor u-comment h-cite' if is_successor
+    classes << 'entry-center h-entry' if include_threads
     classes.join(' ')
   end
 
diff --git a/app/views/stream_entries/_status.html.haml b/app/views/stream_entries/_status.html.haml
index 29dcd6081..f70e2c890 100644
--- a/app/views/stream_entries/_status.html.haml
+++ b/app/views/stream_entries/_status.html.haml
@@ -3,22 +3,20 @@
 - is_successor    ||= false
 - centered        ||= include_threads && !is_predecessor && !is_successor
 
-%div{ class: [is_predecessor ? 'u-in-reply-to h-cite' : nil, is_successor ? 'u-comment h-cite' : nil, !is_predecessor && !is_successor ? 'h-entry' : nil].compact }
-  - if status.reply? && include_threads
-    = render partial: 'status', collection: @ancestors, as: :status, locals: { is_predecessor: true }
+- if status.reply? && include_threads
+  = render partial: 'status', collection: @ancestors, as: :status, locals: { is_predecessor: true }
 
-  .entry{ class: entry_classes(status, is_predecessor, is_successor, include_threads) }
-    - if status.reblog?
-      .pre-header
-        %div.pre-header__icon
-          = fa_icon('retweet fw')
-        %span
-          = link_to TagManager.instance.url_for(status.account), class: 'status__display-name muted' do
-            %strong= display_name(status.account)
-          = t('stream_entries.reblogged')
+.entry{ class: entry_classes(status, is_predecessor, is_successor, include_threads) }
+  - if status.reblog?
+    .pre-header
+      %div.pre-header__icon
+        = fa_icon('retweet fw')
+      %span
+        = link_to TagManager.instance.url_for(status.account), class: 'status__display-name muted' do
+          %strong= display_name(status.account)
+        = t('stream_entries.reblogged')
 
-    %div{ class: status.reblog? ? 'u-repost-of h-cite' : nil }
-      = render partial: centered ? 'stream_entries/detailed_status' : 'stream_entries/simple_status', locals: { status: proper_status(status) }
+  = render partial: centered ? 'stream_entries/detailed_status' : 'stream_entries/simple_status', locals: { status: proper_status(status) }
 
-  - if include_threads
-    = render partial: 'status', collection: @descendants, as: :status, locals: { is_successor: true }
+- if include_threads
+  = render partial: 'status', collection: @descendants, as: :status, locals: { is_successor: true }