Implement programme status tallies for the new session overview page #4639
Conversation
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
jhenderson
changed the title
jhenderson
force-pushed
the
new-session-overview-layout-programme-status-tallies
branch
5 times, most recently
from
4d6b45d to
dde5aed
Compare
jhenderson
commented
Sep 17, 2025
jhenderson
force-pushed
the
new-session-overview-layout-programme-status-tallies
branch
5 times, most recently
from
ea95e76 to
9d192bb
Compare
paulrobertlloyd
requested changes
Sep 18, 2025
| @include nhsuk-font(16, $weight: bold); | ||
| } | ||
|
|
||
| .app-card__heading { |
There was a problem hiding this comment.
These changes will affect other cards… I thought I’d already made these changes, but evidently not. Probably worth me creating a separate PR to handle how we handle styling featured card headings versus entire cards.
There was a problem hiding this comment.
Addressed in #4671
jhenderson
force-pushed
the
new-session-overview-layout-programme-status-tallies
branch
2 times, most recently
from
89e83c3 to
534d6c3
Compare
jhenderson
force-pushed
the
new-session-overview-layout-programme-status-tallies
branch
from
534d6c3 to
bcc8d52
Compare
The session overview page is getting a new design layout which focuses on giving greater details about the progress of a session using tallies across various metrics such Eligible cohort, Vaccinated, Could not vaccinate etc. The overview page no is longer divided into 2 columns, and instead split up into 5 sections: 1. Programme status 2. Consent status 3. Session dates 4. Action required 5. About the location This commit will start with Programme status. The new design will only activate if the URL has the query param "tallies=true". This gives the testers are chance to see and understand the new design as it's built out.
jhenderson
force-pushed
the
new-session-overview-layout-programme-status-tallies
branch
from
bcc8d52 to
ea23dd7
Compare
|
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.
Description
The session overview page is getting a new design layout which focuses on giving greater details about the progress of a session using tallies across various metrics such Eligible cohort, Vaccinated, Could not vaccinate etc.
Jira Ticket - MAV-1982
The overview page no is longer divided into 2 columns, and instead split up into 5 sections:
This commit will start with Programme status. The new design will only activate if the URL has the query param
"tallies=true". This gives the testers are chance to see and understand the new design as it's built out.Screenshot
Note: Eligible cohort should equal Vaccinated + Could not vaccinate + No outcome. The numbers don't add up in the screenshot, probably due discrepancies with the seed data. This screen is for illustration only.