Merge pull request consuldemocracy#2397 from consul/hide_order_select…

…or_if_only_one_order Hide order selectors when only one order available
CDJ11 · Jan 30, 2018 · c06276b · c06276b
2 parents a159e17 + ee7ba46
commit c06276b
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 36 deletions.
diff --git a/app/views/shared/_order_selector.html.erb b/app/views/shared/_order_selector.html.erb
@@ -1,14 +1,16 @@
-<form class="inline-block">
-  <label for="order-selector-participation" class="show-for-sr"><%= t("#{i18n_namespace}.select_order") %></label>
-  <select class="js-location-changer js-order-selector select-order"
-          data-order="<%= @current_order %>"
-          name="order-selector"
-          id="order-selector-participation">
-     <% @valid_orders.each do |order| %>
-       <option <%= 'selected' if order == @current_order %>
-          value='<%= current_path_with_query_params(order: order, page: 1) %>'>
-         <%= t("#{i18n_namespace}.orders.#{order}") %>
-       </option>
-     <% end %>
-   </select>
-</form>
+<% if @valid_orders.present? && @valid_orders.count > 1 %>
+  <form class="inline-block">
+    <label for="order-selector-participation" class="show-for-sr"><%= t("#{i18n_namespace}.select_order") %></label>
+    <select class="js-location-changer js-order-selector select-order"
+            data-order="<%= @current_order %>"
+            name="order-selector"
+            id="order-selector-participation">
+       <% @valid_orders.each do |order| %>
+         <option <%= 'selected' if order == @current_order %>
+            value='<%= current_path_with_query_params(order: order, page: 1) %>'>
+           <%= t("#{i18n_namespace}.orders.#{order}") %>
+         </option>
+       <% end %>
+     </select>
+  </form>
+<% end %>
diff --git a/app/views/shared/_wide_order_selector.html.erb b/app/views/shared/_wide_order_selector.html.erb
@@ -2,25 +2,26 @@
    #
    #   i18n_namespace: for example "moderation.debates.index"
 %>
-
-<div class="wide-order-selector small-12 medium-8">
-  <form>
-    <div class="small-12 medium-6 float-left">
-      <label for="order-selector-participation">
-        <%= t("#{i18n_namespace}.select_order") %>
-      </label>
-    </div>
-    <div class="small-12 medium-6 float-left">
-      <select class="js-location-changer js-order-selector select-order"
-              data-order="<%= @current_order %>" name="order-selector"
-              id="order-selector-participation">
-        <% @valid_orders.each do |order| %>
-          <% value = current_path_with_query_params(order: order, page: 1) %>
-          <option value="<%= value %>" <%= 'selected' if order == @current_order %>>
-            <%= t("#{i18n_namespace}.orders.#{order}") %>
-          </option>
-        <% end %>
-      </select>
-    </div>
-  </form>
-</div>
+<% if @valid_orders.present? && @valid_orders.count > 1 %>
+  <div class="wide-order-selector small-12 medium-8">
+    <form>
+      <div class="small-12 medium-6 float-left">
+        <label for="order-selector-participation">
+          <%= t("#{i18n_namespace}.select_order") %>
+        </label>
+      </div>
+      <div class="small-12 medium-6 float-left">
+        <select class="js-location-changer js-order-selector select-order"
+                data-order="<%= @current_order %>" name="order-selector"
+                id="order-selector-participation">
+          <% @valid_orders.each do |order| %>
+            <% value = current_path_with_query_params(order: order, page: 1) %>
+            <option value="<%= value %>" <%= 'selected' if order == @current_order %>>
+              <%= t("#{i18n_namespace}.orders.#{order}") %>
+            </option>
+          <% end %>
+        </select>
+      </div>
+    </form>
+  </div>
+<% end %>