Skip to content

Hide Share button on Hosted Gallery - #16139

Merged
dskamiotis merged 1 commit into
mainfrom
ds/hide-share-button-from-hosted-gallery
Jun 15, 2026
Merged

dskamiotis merged 1 commit into
mainfrom
ds/hide-share-button-from-hosted-gallery

Conversation

@dskamiotis

@dskamiotis dskamiotis commented Jun 11, 2026 •

Copy link
Copy Markdown
Contributor

What does this change?

Hides the Share button in image captions for Hosted Gallery, while keeping it visible for non-Hosted Gallery types (including Labs Gallery).

Why?

As part of the Hosted Content redesign, Hosted Gallery pages reuse the Gallery/Labs Gallery body structure, which includes image captions with a Share button.
For Hosted Gallery pages, this caption-level Share button is not required.
This change removes it only for Hosted Gallery and preserves existing behaviour elsewhere.

Note: This is an isolated PR created from branch cemms1/hosted-gallery so Hosted Gallery behaviour can be tested before that work is in production.
The change is intentionally scoped to Hosted Gallery logic only and does not affect current production Labs Gallery behaviour.

Screenshots

Tested on Hosted Gallery using branch cemms1/hosted-gallery

Before After
Screenshot 2026-06-11 at 10 41 53 Screenshot 2026-06-11 at 10 41 30

The current behaviour for Labs Gallery which still need the share button are successfully unaffected

Labs Gallery still has share button
Screenshot 2026-06-11 at 10 42 42

@dskamiotis dskamiotis added run_chromatic Runs chromatic when label is applied feature Departmental tracking: work on a new feature labels Jun 11, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jun 11, 2026
@dskamiotis
dskamiotis requested a review from a team June 11, 2026 19:54
@dskamiotis dskamiotis changed the title Hide Share button on Hosted Gallery articles Hide Share button on Hosted Gallery Jun 11, 2026

@deedeeh deedeeh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good @dskamiotis 👍🏼

}
/>
</Island>
{shouldIncludeShareButton && (

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think it would make sense to put this outside of the div element above it, because at the moment if the share button is omitted you still have the padding-top value, even without anything rendering

@dskamiotis
dskamiotis merged commit 9f5e873 into main Jun 15, 2026
35 of 36 checks passed
@dskamiotis
dskamiotis deleted the ds/hide-share-button-from-hosted-gallery branch June 15, 2026 10:04
@gu-prout

gu-prout Bot commented Jun 15, 2026

Copy link
Copy Markdown

Seen on PROD (merged by @dskamiotis 9 minutes and 14 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.

3 participants