Merge branch 'feature/category-page-featured-images' into 'release'

Add option to display images for first three pages in category page

See merge request buckinghamshire-council/bc!737

CHANGELOG.md

@@ -5,6 +5,7 @@
 - EHCCo widget: Update widget heading text
 - EHCCo widget: Make emails clickable by converting them to links
 - EHCCo widget: Change fallback text from TBC to awaiting allocation
+- Add option to display images for first three pages in category page
 
 ## 78.2 (2024-08-14)
 

bc/family_information/blocks.py

@@ -9,6 +9,9 @@ def get_context(self, value, parent_context=None):
 
         # Get the page's class.
         context["page_class"] = parent_context["page"]._meta.object_name
+        context["display_featured_images"] = getattr(
+            parent_context["page"], "display_featured_images", False
+        )
 
         # Check if the block is preceded by a heading.
         blocks_under_headings = parent_context.get("blocks_under_headings", [])

bc/family_information/migrations/0017_categorypage_display_featured_images_and_more.py

@@ -0,0 +1,31 @@
+# Generated by Django 4.2.14 on 2024-08-13 08:09
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        (
+            "family_information",
+            "0016_directory_search_block__add_bucks_online_directory_option",
+        ),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="categorypage",
+            name="display_featured_images",
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name="categorytypeonepage",
+            name="display_featured_images",
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name="categorytypetwopage",
+            name="display_featured_images",
+            field=models.BooleanField(default=False),
+        ),
+    ]

bc/family_information/models.py

@@ -228,6 +228,8 @@ class BaseCategoryPage(FISBannerFields, BasePage):
         default="What do you want to do?", max_length=255
     )
 
+    display_featured_images = models.BooleanField(default=False)
+
     # Content
     body = StreamField(
         [
@@ -262,6 +264,7 @@ class BaseCategoryPage(FISBannerFields, BasePage):
     content_panels = (
         BasePage.content_panels
         + [
+            FieldPanel("display_featured_images"),
             MultiFieldPanel(
                 [
                     FieldPanel("top_tasks_heading", heading="Heading"),
@@ -301,6 +304,10 @@ def allowed_subpage_models(cls):
             resolve_model_string("standardpages.IndexPage")
         ]
 
+    @cached_property
+    def has_featured_pages(self):
+        return any(row.block_type == "cards" for row in self.body)
+
     @cached_property
     def other_child_pages(self):
         """Get child pages for the current category page, excluding children that are

bc/project_styleguide/templates/patterns/molecules/streamfield/blocks/cards_block.html

@@ -5,7 +5,11 @@
         data-height-equalizer
     >
         {% for card in value %}
-            {% include "patterns/molecules/page-card/page-card.html" with classes="card--fis-index" has_heading=has_heading page=card.value.specific %}
+            {% if display_featured_images and forloop.counter <= 3 %}
+                {% include "patterns/molecules/page-card/page-card.html" with classes="card--fis-index" has_heading=has_heading page=card.value.specific show_image=True %}
+            {% else %}
+                {% include "patterns/molecules/page-card/page-card.html" with classes="card--fis-index" has_heading=has_heading page=card.value.specific show_image=False %}
+            {% endif %}
         {% endfor %}
     </section>
 {% endif %}

bc/project_styleguide/templates/patterns/pages/standardpages/index_page--fis-cat1.html

@@ -48,7 +48,11 @@ <h2 class="heading heading--l" id="{{ page.other_pages_heading|slugify }}">{{ pa
             data-height-equalizer
         >
             {% for subpage in page.other_child_pages %}
-                {% include "patterns/molecules/page-card/page-card.html" with classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                {% if not page.has_featured_pages and page.display_featured_images and forloop.counter <= 3 %}
+                    {% include "patterns/molecules/page-card/page-card.html" with show_image=True classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                {% else %}
+                    {% include "patterns/molecules/page-card/page-card.html" with show_image=False classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                {% endif %}
             {% endfor %}
         </section>
     </div>

bc/project_styleguide/templates/patterns/pages/standardpages/index_page--fis-cat2.html

@@ -49,7 +49,11 @@ <h2 class="heading heading--l" id="{{ page.other_pages_heading|slugify }}">{{ pa
                     data-height-equalizer
                 >
                     {% for subpage in page.other_child_pages %}
-                        {% include "patterns/molecules/page-card/page-card.html" with classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                        {% if not page.has_featured_pages and page.display_featured_images and forloop.counter <= 3 %}
+                            {% include "patterns/molecules/page-card/page-card.html" with show_image=True classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                        {% else %}
+                            {% include "patterns/molecules/page-card/page-card.html" with show_image=False classes="card--fis-index" has_heading=show_other_pages_heading page=subpage %}
+                        {% endif %}
                     {% endfor %}
                 </div>
             </section>