Merge pull request #2686 from tvdeyen/fix-picture-archive-empty-state

Fix layout of empty picture archive

app/views/alchemy/admin/pictures/_archive.html.erb

@@ -38,7 +38,6 @@
       style: 'float: none'
     ) %>
   </div>
-  <div id="pictures" class="picture-size--<%= @size %>">
   <% if @pictures.blank? and @recent_pictures.blank? and search_filter_params[:q].blank? %>
     <%= render_message do %>
       <%= Alchemy.t(:no_images_in_archive) %>
@@ -48,9 +47,10 @@
       <%= Alchemy.t(:no_search_results) %>
     <% end %>
   <% else %>
-    <%= render partial: 'picture', collection: @pictures %>
+    <div id="pictures" class="picture-size--<%= @size %>">
+      <%= render partial: 'picture', collection: @pictures %>
+    </div>
   <% end %>
-  </div>
 <% end %>
 
 <%= paginate @pictures, theme: 'alchemy' %>

app/views/alchemy/admin/pictures/_archive_overlay.html.erb

@@ -8,7 +8,20 @@
       <%= render 'tag_list' %>
     </div>
   </div>
-  <div id="overlay_picture_list" class="picture-size--<%= @size %>">
-    <%= render 'overlay_picture_list' %>
-  </div>
+  <% if @pictures.empty? %>
+    <%= render_message do %>
+      <% if search_filter_params.empty? %>
+        <%= Alchemy.t(:no_images_in_archive) %>
+      <% else %>
+        <%= Alchemy.t(:no_search_results) %>
+      <% end %>
+    <% end %>
+  <% else %>
+    <div id="overlay_picture_list" class="picture-size--<%= @size %>">
+      <%= render partial: 'picture_to_assign',
+        collection: @pictures,
+        locals: {size: @size} %>
+      <%= paginate @pictures, theme: 'alchemy', remote: true, hide_per_page_select: true %>
+    </div>
+  <% end %>
 </div>