Skip to content

Fix the broken headline due to main media bug#14376

Merged
marjisound merged 1 commit intomainfrom
fix-gallery-main-media-bug
Aug 11, 2025
Merged

Fix the broken headline due to main media bug#14376
marjisound merged 1 commit intomainfrom
fix-gallery-main-media-bug

Conversation

@marjisound
Copy link
Contributor

@marjisound marjisound commented Aug 11, 2025

Co-authored-by: Dina Hafez dina.hafez@guardian.co.uk

What does this change?

This PR fixes a bug that was introduced in the previous gallery PR #14163

We don't need a position: relative for the parent of LightboxLink in main media because the button will be invisible for main media in Galleries.

Why?

Screenshots

Before After
before after

Co-authored-by: Dina Hafez <dina.hafez@guardian.co.uk>
@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.

Copy link
Contributor

@deedeeh deedeeh left a comment

Choose a reason for hiding this comment

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

Looks much better now 👍🏼

@marjisound marjisound added the run_chromatic Runs chromatic when label is applied label Aug 11, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Aug 11, 2025
@marjisound marjisound merged commit 21b5fd0 into main Aug 11, 2025
32 checks passed
@marjisound marjisound deleted the fix-gallery-main-media-bug branch August 11, 2025 15:33
@prout-bot
Copy link

Seen on PROD (merged by @marjisound 18 minutes and 54 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.

3 participants