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

feat: minor styling improvements #240

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

pieterjanscheir
Copy link

@pieterjanscheir pieterjanscheir commented Apr 30, 2024

Description

This PR fixes some minor styling on the public site.
-->

What type of PR is this? (check all applicable)

  • πŸ’‘ Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

N/A

Mobile & Desktop Screenshots/Recordings

image
The bottom shadow of the cards is now not cut-off by missing padding.

image
Also on mobile, the "as seen on twitter" doesn't stick to the header.

Steps to QA

  1. Open the site and see the changes.

Added to documentation?

  • πŸ“œ README.md
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

=> no

[optional] What gif best describes this PR or how it makes you feel?

I just contributed some small styling improvements. Just trying to be helpful.

Copy link

vercel bot commented Apr 30, 2024

@pieterjanscheir is attempting to deploy a commit to the Salgsmaskin Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Owner

@Codehagen Codehagen 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 ✨

@Codehagen Codehagen merged commit 8b84a7b into Codehagen:main Apr 30, 2024
1 check failed
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.

2 participants