Skip to content

Add gallery main media caption#14261

Merged
marjisound merged 3 commits intomainfrom
add-main-media-caption
Jul 23, 2025
Merged

Add gallery main media caption#14261
marjisound merged 3 commits intomainfrom
add-main-media-caption

Conversation

@marjisound
Copy link
Contributor

@marjisound marjisound commented Jul 22, 2025

What does this change?

This PR adds main media caption for Gallery articles

Fixes part of #12648

@github-actions
Copy link

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.

@github-actions
Copy link

github-actions bot commented Jul 22, 2025

@marjisound marjisound linked an issue Jul 22, 2025 that may be closed by this pull request
@marjisound marjisound changed the title Add main media caption Add gallery main media caption Jul 22, 2025
@marjisound marjisound self-assigned this Jul 22, 2025
@marjisound marjisound added the run_chromatic Runs chromatic when label is applied label Jul 22, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jul 22, 2025
@github-actions
Copy link

github-actions bot commented Jul 22, 2025

@marjisound marjisound requested review from a team and JamieB-gu July 23, 2025 08:21
@marjisound marjisound added the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@marjisound marjisound force-pushed the add-main-media-caption branch from 459c645 to 382b79a Compare July 23, 2025 08:38
@marjisound marjisound added the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
Comment on lines +160 to +170
css`
${grid.column.centre}
${from.leftCol} {
${grid.column.left}
}
position: relative;
${between.tablet.and.leftCol} {
&::before {
content: '';
position: absolute;
left: -10px;
top: 0;
bottom: 0;
width: 1px;
background-color: ${palette(
'--article-border',
)};
}
}
`,
]}
css={[grid.between('centre-column-start', 'grid-end')]}
>
Main media caption
Meta
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need a placeholder for the meta section? It looks like this section has already been implemented above?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, just removed it 👍

@marjisound marjisound added the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jul 23, 2025
@marjisound marjisound merged commit f66c847 into main Jul 23, 2025
31 checks passed
@marjisound marjisound deleted the add-main-media-caption branch July 23, 2025 17:30
@prout-bot
Copy link

Seen on PROD (merged by @marjisound 11 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Gallery Main Media Component

3 participants