UI improvements in subscribers/group/roles filtering #2615
Conversation
amaierhofer
commented
May 15, 2024
amaierhofer
commented
- bigger group names
- visually separate groups
- do not render layer without subgroups
amaierhofer
force-pushed
the
bug/2611-visual-improvements-to-subscriber-roles-filter
branch
from
7a92671
to
1232ab5
Compare
| @@ -5,9 +5,9 @@ | |||
|
|
|||
| - if @role_types | |||
| - @role_types.each do |layer, groups| | |||
| %h4.filter-toggle=layer | |||
| %h3.filter-toggle.mt-4=layer if groups.any? | |||
There was a problem hiding this comment.
In welchem Fall ist dieses if nützlich? Man muss doch immer zuerst mit dem Typeahead eine Gruppe auswählen, und erst dann wird dieser Abschnitt überhaupt angezeigt (via if @role_types zwei Zeilen weiter oben?)
Präsenz von @role_types impliziert meiner Ansicht nach groups.any?, sehe ich das falsch? Du schreibst "do not render layer without subgroups", aber es kann ja auch ein Layer mit Rollen aber ohne Subgruppen geben..?
There was a problem hiding this comment.
Ja, es wird immer zuerst eine Gruppe ausgewählt und dann entsprechend die @role_types befüllt.
Falls jetzt der Layer groupen mit Rollen enthält (weil dieses als global betrachtet werden wie zb beim Sac) dann erscheinen aktuell nur die Layer als überschriften aber ohne auswählbare Rollen, siehe screenshot
Habe die Anpassung auch noch im role partial beim people_filter nachgezogen
- bigger group names (roles) - visually separate groups (roles) - do not render layer without subgroups (roles) - proper spacing for tags
carlobeltrame
force-pushed
the
bug/2611-visual-improvements-to-subscriber-roles-filter
branch
from
04c6769
to
e39479c
Compare
carlobeltrame
deleted the
bug/2611-visual-improvements-to-subscriber-roles-filter
branch
