Skip to content

Cricket match header result#15824

Merged
Jakeii merged 9 commits into
mainfrom
jlk/cricket-match-header-result
May 12, 2026
Merged

Cricket match header result#15824
Jakeii merged 9 commits into
mainfrom
jlk/cricket-match-header-result

Conversation

@Jakeii
Copy link
Copy Markdown
Member

@Jakeii Jakeii commented May 7, 2026

closes #15684

What does this change?

Add types and UI for the cricket match result based on the data from PA, most of the time there is a nice description like "Australia win by an innings and 47 runs" which we can use, but sometimes there isn't so we can fallback to one constructed from the data available.

I've used the same style as the "comment" from the football header which is used for football results:

Screenshot 2026-05-07 at 17 17 11

This is different from the original idea of the winning score at the bottom of the team column as cricket results can be varied and complex:

Image

Examples I've come across already

  • "Match Tied (South Africa win the second Super Over)"
  • "New Zealand win by 6 wickets (DLS Method)"
  • "Match abandoned without a ball bowled"

Screenshots

Before After
before after
before2 after2

@Jakeii Jakeii force-pushed the jlk/cricket-match-header-result branch from 3a0081e to 4d6222c Compare May 7, 2026 15:55
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

@Jakeii Jakeii added the feature Departmental tracking: work on a new feature label May 7, 2026
@Jakeii Jakeii added the run_chromatic Runs chromatic when label is applied label May 7, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label May 7, 2026
@Jakeii Jakeii marked this pull request as ready for review May 8, 2026 08:44
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@Jakeii Jakeii requested a review from a team May 8, 2026 08:47
@Jakeii Jakeii merged commit 8c39907 into main May 12, 2026
31 checks passed
@Jakeii Jakeii deleted the jlk/cricket-match-header-result branch May 12, 2026 11:15
@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented May 12, 2026

Seen on PROD (merged by @Jakeii 12 minutes and 8 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show winning team in cricket match header

2 participants