Skip to content

Remove isVideoArticle prop from youtubeAtomFeatureCard #15307

Open
abeddow91 wants to merge 1 commit intomainfrom
ab/youtube-overlay-prop
Open

Remove isVideoArticle prop from youtubeAtomFeatureCard #15307
abeddow91 wants to merge 1 commit intomainfrom
ab/youtube-overlay-prop

Conversation

@abeddow91
Copy link
Contributor

What does this change?

Remove isVideoArticle prop from youtubeAtomFeatureCard as there is access to format in this component.

This means we can decide if its a video article within the component.

Why?

Reduce the number of props and the amount of prop drilling.

Screenshots

This is a no-op change.

…access to format in this component.

This means we can decide if its a video article within the component and reduce prop drilling
@abeddow91 abeddow91 requested a review from domlander February 5, 2026 16:44
@abeddow91 abeddow91 self-assigned this Feb 5, 2026
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

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.

@abeddow91 abeddow91 added fronts + curation maintenance Departmental tracking: maintenance work, not a fix or a feature labels Feb 5, 2026
@domlander
Copy link
Contributor

Nice improvement!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fronts + curation 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