Increase homepage sector cards from 3 to 4 - #446
Merged
jhancock532 merged 2 commits intoJul 29, 2026
Conversation
jhancock532
force-pushed
the
chore/update-homepage-division-signpost-block
branch
from
July 27, 2026 10:22
c5018ae to
72c95b0
Compare
jhancock532
marked this pull request as ready for review
July 27, 2026 10:26
MaciekBaron
reviewed
Jul 29, 2026
helenb
approved these changes
Jul 29, 2026
helenb
left a comment
Member
There was a problem hiding this comment.
I thought I had found a bug but it seemed to be related to missing images. I will approve but please could you complete the A11y tests from the PR checklist, and if time update the styleguide so that there are images displayed for the sector cards.
Author
|
Ran through the accessibility checks following |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There was no ticket for this work, context provided by Lily is as follows.
Summary
The homepage currently limits the sector cards component to a maximum of 3 items. As part of the new IA, we need to increase this to 4.
Background
We're removing the Wagtail CMS card and replacing the current three-card layout with four sectors:
At the moment, the card block in Wagtail has a hard limit of three items, preventing us from adding the fourth sector.
Acceptance criteria
Description of Changes Made
How to Test
Add four cards to the homepage, and verify they render well across viewports.
Check out the pattern library component locally: http://localhost:8000/pattern-library/render-pattern/patterns/molecules/streamfield/blocks/division_signpost_block.html
Screenshots
Expand to see more
Light mode, with focus state, on desktop.
Dark mode, with focus state, on tablet - desktop.
Safari - external monitor
MR Checklist
Unit tests
Documentation
Browser testing
Data protection
Light and dark mode
Accessibility
Sustainability
Pattern library