Fix accessibility community page and add empty case

django · Nov 25, 2023 · 4aa4c97 · 4aa4c97
1 parent 710e8b4
commit 4aa4c97
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 7 deletions.
diff --git a/djangoproject/scss/_dark-mode.scss b/djangoproject/scss/_dark-mode.scss
@@ -57,6 +57,9 @@ html[data-theme="light"],
     --search-mark-text: #{$green-dark};
 
     --selection: #{$green-very-light};
+
+    --community-img-bg: #{$green-very-light};
+    --community-img-fg: #{$green-dark};
 }
 
 @media (prefers-color-scheme: dark) {
@@ -105,6 +108,9 @@ html[data-theme="light"],
         --search-mark-text: #{$black-light-5};
 
         --selection: #{$green-medium};
+
+        --community-img-bg: #{$green-dark};
+        --community-img-fg: #{$white};
     }
 
     body .homepage {
@@ -201,6 +207,9 @@ html[data-theme="dark"] {
 
     --selection: #{$green-dark};
 
+    --community-img-bg: #{$green-dark};
+    --community-img-fg: #{$white};
+
     .img-release {
         filter: invert(1);
     }

diff --git a/djangoproject/scss/_style.scss b/djangoproject/scss/_style.scss
@@ -3685,11 +3685,11 @@ ul.corporate-members li {
         flex-direction: column;
         justify-content: center;
         align-items: center;
-        background: var(--secondary-accent);
+        background: var(--community-img-bg);
         border-radius: 20px;
     }
 
-    .community-cta svg {
-        color: var(--body-fg);
+    .community-cta svg, h3 {
+        color: var(--community-img-fg);
     }
 }
diff --git a/djangoproject/templates/aggregator/local-django-community.html b/djangoproject/templates/aggregator/local-django-community.html
@@ -8,7 +8,7 @@
 
   <h2>Local Django Communities</h2>
 
-  <h3>Table of contents<a class="plink" href="#table-of-contents">¶</a></h3>
+  {% if grouped_django_communities %}<h3>Table of contents<a class="plink" href="#table-of-contents"> ¶</a></h3>{% endif %}
   <ul>
     {% for local_django_community in grouped_django_communities %}
       <li><a href="#{{ local_django_community.grouper.title }}-meetups">{{ local_django_community.grouper.title }}</a></li>
@@ -18,11 +18,11 @@ <h3>Table of contents<a class="plink" href="#table-of-contents">¶</a></h3>
 
   {% for local_django_community in grouped_django_communities %}
     <div class="section">
-      <h2>{{ local_django_community.grouper.title }}<a class="plink" href="#{{ local_django_community.grouper.title }}-meetups">¶</a></h2>
-      <ul style="list-style-type:none;">
+      <h2>{{ local_django_community.grouper.title }} <a class="plink" href="#{{ local_django_community.grouper.title | lower }}-meetups">¶</a></h2>
+      <ul>
         {% for django_community in local_django_community.list %}
           <li>
-            <h3 id="{{ django_community.slug }}-team">{{ django_community.name }}<a class="plink" href="#{{ django_community.slug }}-meetup">¶</a></h3>
+            <h3 id="{{ django_community.slug }}-team">{{ django_community.name }} <a class="plink" href="#{{ django_community.slug }}-meetup">¶</a></h3>
             <p class="meta">{{ django_community.city }}, {{ django_community.country }} &nbsp;
               {% if django_community.is_active %}
                 Active
@@ -43,6 +43,8 @@ <h3 id="{{ django_community.slug }}-team">{{ django_community.name }}<a class="p
         {% endfor %}
       </ul>
     </div>
+  {% empty %}
+  Local Django communities are coming soon. Please check back later.
 
   {% endfor %}
 {##}