Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CardHeader used instead of CardFooter #45585

Merged
merged 2 commits into from
Dec 22, 2023
Merged

Conversation

gvgvgvijayan
Copy link
Contributor

What?

CardHeader used instead of CardFooter at the last sub/child component of Card component

Why?

Give correct sample of Card and it's sub component usage.

@codesandbox
Copy link

codesandbox bot commented Nov 7, 2022

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

@carolinan carolinan added the [Type] Developer Documentation Documentation for developers label Nov 9, 2022
Copy link
Member

@skorasaurus skorasaurus left a comment

Choose a reason for hiding this comment

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

looks good to me; thanks.

@gvgvgvijayan
Copy link
Contributor Author

Gentle reminder about this PR not yet merged. Please share if anything need from my side. Thanks.

@ndiego ndiego enabled auto-merge (squash) April 21, 2023 14:19
@ryanwelcher ryanwelcher disabled auto-merge June 6, 2023 04:51
@ryanwelcher ryanwelcher enabled auto-merge (squash) June 6, 2023 04:52
@annezazu
Copy link
Contributor

annezazu commented Jun 7, 2023

Hey there. Can you try merging in latest trunk? Right now, some of the automated items have failed/didn't report and it's making it so it can't be merged.

@gvgvgvijayan
Copy link
Contributor Author

@annezazu synced with latest trunk, thanks.

@t-hamano t-hamano closed this Dec 22, 2023
@t-hamano t-hamano reopened this Dec 22, 2023
@t-hamano t-hamano added the [Package] Components /packages/components label Dec 22, 2023
@t-hamano
Copy link
Contributor

GitHub Action seemed to have stopped, but after closing and reopening this PR, it seems to have started working again. I also think this change makes sense, so I'm merging it.

@gvgvgvijayan Thanks for your contribution!

@t-hamano t-hamano merged commit 1c53d8b into WordPress:trunk Dec 22, 2023
100 of 103 checks passed
@github-actions github-actions bot added this to the Gutenberg 17.4 milestone Dec 22, 2023
@gvgvgvijayan
Copy link
Contributor Author

Thanks for rechecking @t-hamano 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Components /packages/components [Type] Developer Documentation Documentation for developers
5 participants