diff --git a/app/assets/stylesheets/users.css b/app/assets/stylesheets/users.css new file mode 100644 index 000000000..e3e427779 --- /dev/null +++ b/app/assets/stylesheets/users.css @@ -0,0 +1,4 @@ +.users-list { + max-block-size: 40dvh; + overflow: auto; +} diff --git a/app/assets/stylesheets/utilities.css b/app/assets/stylesheets/utilities.css index 6ec13a630..f80bbd988 100644 --- a/app/assets/stylesheets/utilities.css +++ b/app/assets/stylesheets/utilities.css @@ -107,6 +107,7 @@ .pad-inline-double { padding-inline: var(--inline-space-double); } .unpad { padding: 0; } + .unpad-block-end { padding-block-end: 0; } .unpad-inline { padding-inline: 0; } /* Margins */ diff --git a/app/helpers/accesses_helper.rb b/app/helpers/accesses_helper.rb index a139ac3d4..8715b8165 100644 --- a/app/helpers/accesses_helper.rb +++ b/app/helpers/accesses_helper.rb @@ -1,6 +1,6 @@ module AccessesHelper def access_menu_tag(collection, &) - tag.menu class: [ "flex flex-column gap margin-none pad txt-medium", { "toggler--toggled": collection.all_access? } ], data: { + tag.menu class: [ "flex flex-column gap-half unpad margin-none txt-medium", { "toggler--toggled": collection.all_access? } ], data: { controller: "filter toggle-class", filter_active_class: "filter--active", filter_selected_class: "selected", toggle_class_toggle_class: "toggler--toggled" }, & diff --git a/app/views/collections/_access_toggle.erb b/app/views/collections/_access_toggle.erb index 318415703..e08f1dc85 100644 --- a/app/views/collections/_access_toggle.erb +++ b/app/views/collections/_access_toggle.erb @@ -11,7 +11,7 @@ - <%= icon_tag "check", class: "toggler__visible-when-on" %> + <%= icon_tag "check", class: "toggler__visible-when-on margin-inline-end" %> -
+
<%= access_menu_tag @collection do %> -
  • +
  • <%= icon_tag "everyone" %> Everyone @@ -46,14 +46,16 @@
  • - + <% if User.active.count > 20 %> <% end %> -
    - <%= access_toggles_for @selected_users, selected: true %> +
    + <% 20.times do %> + <%= access_toggles_for @selected_users, selected: true %> + <% end %> <% if @selected_users.any? && @unselected_users.any? %>