Merge pull request #753 from euphorie/scrum-2333-certificate-columns-2

Certificates overview: set CSS class according to number of items
euphorie · Jul 17, 2024 · d4a54e7 · d4a54e7
2 parents a62e29f + 0752957
commit d4a54e7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/docs/changes.rst b/docs/changes.rst
@@ -10,6 +10,9 @@ Changelog
 - Fix the last modifier id for guest users that have been converted to registered users
   [ale-rt]
 
+- Adjusted column layout for certificates overview
+  [reinhardt]
+
 
 16.2.0 (2024-06-12)
 -------------------

diff --git a/src/euphorie/client/browser/certificate.py b/src/euphorie/client/browser/certificate.py
@@ -88,3 +88,6 @@ def certificates(self):
                 year_certificates, key=lambda c: c["date"], reverse=True
             )
         return certificates.items()
+
+    def get_num_columns(self, num_items):
+        return 2 if num_items < 3 else 3
diff --git a/src/euphorie/client/browser/templates/certificates.pt b/src/euphorie/client/browser/templates/certificates.pt
@@ -25,11 +25,12 @@
                 <tal:year tal:define="
                             year python:item[0];
                             sessions python:item[1];
+                            num_columns python: view.get_num_columns(len(sessions));
                           ">
                   <h2 class="separation-title">
                   ${year}
                   </h2>
-                  <div class="certificate-overview col-3">
+                  <div class="certificate-overview col-${num_columns}">
                     <tal:session tal:repeat="info sessions">
                       <figure class="certificate-preview">
                         <a class="certificate pat-auto-scale pat-inject"