Clean up h-card mess of divs

diff --git a/app/helpers/stream_entries_helper.rb b/app/helpers/stream_entries_helper.rb
index ae2f575b511149edfdccb5475a6c6e45c144ba31..15601a0796bde4fd997591989419ded6ea9f230c 100644 (file)
--- 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 29dcd6081c83a8e0ba71933ae1149abb8dd106d8..f70e2c890294301a0db08aa2b13b98bd1c0fadfa 100644 (file)
--- 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 }