Skip to content

Conversation

@Haberkamp
Copy link
Contributor

@Haberkamp Haberkamp commented Nov 18, 2025

What?

Removes border-top from card footer.

Why?

Border currently invisible because of wrong css variable assignment anyway.

How?

Removed border-top.

Testing?

The visual tests should be enough

@vercel
Copy link

vercel bot commented Nov 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
meteor-component-library Ready Ready Preview Comment Dec 5, 2025 0:13am

Copy link
Contributor

@sydinh sydinh left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@fabianhueske fabianhueske force-pushed the add-missing-card-footer-border branch from a29b73b to fccacca Compare December 4, 2025 13:16
@fabianhueske fabianhueske changed the title fix(component-library): add border to card footer fix(component-library): fox border-top on card footer Dec 4, 2025
@fabianhueske fabianhueske changed the title fix(component-library): fox border-top on card footer fix(component-library): fix border-top on card footer Dec 4, 2025
@fabianhueske fabianhueske requested a review from sydinh December 4, 2025 13:48
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 4, 2025

Open in StackBlitz

@shopware-ag/meteor-admin-sdk

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/meteor-admin-sdk@969

@shopware-ag/meteor-component-library

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/meteor-component-library@969

@shopware-ag/meteor-icon-kit

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/meteor-icon-kit@969

@shopware-ag/meteor-prettier-config

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/meteor-prettier-config@969

@shopware-ag/stylelint-plugin-meteor

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/stylelint-plugin-meteor@969

@shopware-ag/meteor-tokens

npm i https://pkg.pr.new/shopware/meteor/@shopware-ag/meteor-tokens@969

commit: fccacca

@fabianhueske fabianhueske enabled auto-merge (squash) December 5, 2025 12:12
@fabianhueske fabianhueske merged commit e14e664 into main Dec 5, 2025
8 checks passed
@fabianhueske fabianhueske deleted the add-missing-card-footer-border branch December 5, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants