Merge pull request #162 from springload/fix/group-project-roles

group project roles together
springload · Aug 13, 2024 · 713ffe5 · 713ffe5
2 parents 65ed7b4 + f861de3
commit 713ffe5
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/cdhweb/static_src/global/pages/project-page.scss b/cdhweb/static_src/global/pages/project-page.scss
@@ -86,3 +86,15 @@ the right of the content (when unstacked).
     margin-block-start: 24px;
   }
 }
+
+.project-page__side-content-item--same-role {
+  & + .project-page__side-content-item {
+    margin-block-start: 24px;
+  }
+}
+
+.project-page__side-content-item {
+  & + .project-page__side-content-item--same-role {
+    margin-block-start: 0;
+  }
+}
diff --git a/templates/projects/project_page.html b/templates/projects/project_page.html
@@ -40,8 +40,12 @@ <h2>Links</h2>
                     <h2>Team</h2>
 
                     {% for membership in page.current_memberships %}
-                        <div class="project-page__side-content-item">
-                            <h3>{{ membership.role }}</h3>
+                        {% ifchanged membership.role %}
+                            <div class="project-page__side-content-item">
+                        {% else %}
+                            <div class="project-page__side-content-item--same-role">
+                        {% endifchanged %}
+                            {% ifchanged %}<h3>{{ membership.role }}</h3>{% endifchanged %}
                             <div>
                                 {% include 'includes/project_membership.html' with member=membership.person %}
                             </div>