Allow asides to re-order content/aside so that they can stack in reve…

…rse order

app/models/katalyst/content/aside.rb

@@ -3,6 +3,13 @@
 module Katalyst
   module Content
     class Aside < Layout
+      style_attributes do
+        attribute :reverse, :boolean, default: false
+      end
+
+      def self.permitted_params
+        super + %i[reverse]
+      end
     end
   end
 end

app/views/katalyst/content/asides/_aside.html+form.erb

@@ -22,6 +22,11 @@
     <%= form.check_box :visible %>
   </div>
 
+  <div class="field">
+    <%= form.label :reverse %>
+    <%= form.check_box :reverse %>
+  </div>
+
   <%= form.submit "Done" %>
   <%= link_to "Discard", :back %>
 <% end %>

app/views/katalyst/content/asides/_aside.html.erb

@@ -4,7 +4,7 @@
   <% if aside.children.any? %>
     <% items = aside.children.select(&:visible?) %>
     <% last = items.pop %>
-    <div role="aside-container">
+    <div role="aside-container" <%= "class=reverse" if aside.reverse %>>
       <div>
         <%= render_content_items items %>
       </div>

spec/models/katalyst/content/aside_spec.rb

@@ -4,19 +4,24 @@
 require_relative "item_examples"
 
 RSpec.describe Katalyst::Content::Aside do
-  subject(:aside) { build(:katalyst_content_section, container: page) }
+  subject(:aside) { build(:katalyst_content_aside, container: page) }
 
   let(:page) { create(:page) }
 
   it_behaves_like "a item" do
     let(:item) { aside }
   end
 
+  it "can round trip style attributes" do
+    aside = create(:katalyst_content_aside, container: page, reverse: true)
+    expect(aside.reload).to have_attributes(reverse: true)
+  end
+
   describe "#to_plain_text" do
     it { is_expected.to have_attributes(to_plain_text: aside.heading) }
 
     context "when heading is hidden" do
-      subject(:aside) { build(:katalyst_content_section, heading_style: "none", container: page) }
+      subject(:aside) { build(:katalyst_content_aside, heading_style: "none", container: page) }
 
       it { is_expected.to have_attributes(to_plain_text: nil) }
     end