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

Update card-image.html #1351

Merged
merged 1 commit into from
Nov 10, 2022
Merged

Update card-image.html #1351

merged 1 commit into from
Nov 10, 2022

Conversation

mholt
Copy link
Contributor

@mholt mholt commented Nov 10, 2022

Apply border-radius to left/right card images

Just a very small patch for a very minor detail. I'm using left/right card images and the samples don't have rounded borders, but there are classes for it!

I hope I got the template language correct.

Apply border-radius to left/right card images
@vercel
Copy link

vercel bot commented Nov 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tabler ✅ Ready (Inspect) Visit Preview Nov 10, 2022 at 2:45AM (UTC)

@kevinpapst
Copy link
Collaborator

Thank you very much @mholt 👍

Screenshots are always appreciated to verify your changes.

I guess you changed those card from 90 degree corners:
Bildschirmfoto 2022-11-10 um 11 24 42

to these rounded ones:
Bildschirmfoto 2022-11-10 um 11 24 56

I tried both start-and-end images. Good to go from my side.

@codecalm codecalm merged commit b59e6f9 into tabler:dev Nov 10, 2022
@codecalm
Copy link
Member

thank you! ❤️

@mholt
Copy link
Contributor Author

mholt commented Nov 10, 2022

Yep, that's it.

Thanks, and sorry for the lack of screenshots. I'm a backend guy by trade and this is basically my first contribution to a frontend project. I'll try to do a.screenshot next time. (Although my monitor is low-DPI so I'm worried it'll be fuzzy)

@mholt mholt deleted the patch-1 branch August 22, 2023 19:50
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.

None yet

3 participants