Skip to content

Include CTA button in hosted gallery layout#16198

Open
cemms1 wants to merge 4 commits into
mainfrom
cemms1/include-cta-hosted-gallery
Open

Include CTA button in hosted gallery layout#16198
cemms1 wants to merge 4 commits into
mainfrom
cemms1/include-cta-hosted-gallery

Conversation

@cemms1

@cemms1 cemms1 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

What does this change?

Adds the CTA button from #16047 to the hosted gallery layout

Why?

Adding features to the Hosted Gallery pages as part of the migration from frontend

Screenshots

Before After
before after
before2 after2

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

@cemms1 cemms1 force-pushed the cemms1/include-cta-hosted-gallery branch from 8b66c7b to 9585d92 Compare June 18, 2026 09:27
@cemms1 cemms1 added maintenance Departmental tracking: maintenance work, not a fix or a feature Commercial 💰 run_chromatic Runs chromatic when label is applied labels Jun 18, 2026
@cemms1 cemms1 marked this pull request as ready for review June 18, 2026 09:27
@cemms1 cemms1 requested a review from a team June 18, 2026 09:27
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jun 18, 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.

Nice @cemms1 👍🏼 I left a comment and I can't see CSS changes related to small mobile which I can see in the designs and no screenshots as well. The CTA button and Share should be stacked in small mobile which is from 320px until 375px.

`;

const ctaButtonStyles = css`
margin-right: ${space[1]}px;

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.

Looking at the Figma the spacing between the CTA and the Share button is 12px

Suggested change
margin-right: ${space[1]}px;
margin-right: ${space[3]}px;

@cemms1 cemms1 force-pushed the cemms1/include-cta-hosted-gallery branch 2 times, most recently from 6d9f49d to d628bd0 Compare June 18, 2026 17:11
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

@cemms1 cemms1 force-pushed the cemms1/include-cta-hosted-gallery branch from fd54b8f to d63fd2e Compare June 18, 2026 17:20
@cemms1 cemms1 added the run_chromatic Runs chromatic when label is applied label Jun 18, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jun 18, 2026
@cemms1 cemms1 force-pushed the cemms1/include-cta-hosted-gallery branch from d63fd2e to ccb5029 Compare June 19, 2026 15:26
@cemms1 cemms1 requested a review from a team June 19, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Commercial 💰 maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants