Skip to content

Conversation

@cemms1
Copy link
Contributor

@cemms1 cemms1 commented Sep 17, 2025

What does this change?

Prepare for Labs redesign work by including the switch added in frontend

There should be no visual changes in this PR

Why?

To allow us to develop safely and incrementally while we work on implementing the new designs

@github-actions
Copy link

github-actions bot commented Sep 17, 2025

@cemms1 cemms1 force-pushed the cemms1/prep-for-labs-redesign branch from 510577a to 425cd60 Compare September 17, 2025 14:44
title,
showSectionColours,
)
{!isLabs && !showLabsRedesign && (
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Only including this now as a way to "use" the newly introduced props to avoid linting warnings

if (collection.containerPalette === 'Branded') {
if (
collection.containerPalette === 'Branded' &&
!showLabsRedesign
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We won't render the container redesign here, we'll do it in FrontSection instead. So this will only run if the switch is off

@github-actions
Copy link

github-actions bot commented Sep 17, 2025

@cemms1 cemms1 force-pushed the cemms1/prep-for-labs-redesign branch from 425cd60 to f0a9f33 Compare September 17, 2025 14:46
@cemms1 cemms1 marked this pull request as ready for review September 17, 2025 14:46
@cemms1 cemms1 requested a review from a team September 17, 2025 14:46
@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.

Thanks @cemms1 for doing this to smooth the process of working on Labs redesign ✨

@cemms1 cemms1 added the run_chromatic Runs chromatic when label is applied label Sep 17, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Sep 17, 2025
@cemms1 cemms1 merged commit 3c62208 into main Sep 17, 2025
32 checks passed
@cemms1 cemms1 deleted the cemms1/prep-for-labs-redesign branch September 17, 2025 15:02
@gu-prout
Copy link

gu-prout bot commented Sep 17, 2025

Seen on PROD (merged by @cemms1 7 minutes and 58 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