fix: move margins to button in slideshow

[andrewreifman]

What/Why?

In the Slideshow component, if a slide description is removed, there was no spacing between the title and button. Margin classes were changed to ensure the title, description, and button had the proper spacing between them at all times.

Testing

View the Slideshow component and hide any slide description.

[changeset-bot]

⚠️ No Changeset found

Latest commit: ecaddfc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
catalyst-latest	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 14, 2024 11:12pm

5 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
catalyst	⬜️ Ignored (Inspect)			Nov 14, 2024 11:12pm
catalyst-au	⬜️ Ignored (Inspect)			Nov 14, 2024 11:12pm
catalyst-soul	⬜️ Ignored (Inspect)			Nov 14, 2024 11:12pm
catalyst-uk	⬜️ Ignored (Inspect)			Nov 14, 2024 11:12pm
catalyst-unstable	⬜️ Ignored (Inspect)	Visit Preview		Nov 14, 2024 11:12pm

[jorgemoya]

pnpm-lock shouldn't need to be updated. Component changes look good tho!

[andrewreifman]

pnpm-lock shouldn't need to be updated. Component changes look good tho!

@jorgemoya I assume it changed the lock file by me just running pnpm install. Do I just need to undo that change in the file and push it back up?

I also was about to ask in Slack what I should do about the lint failures in the build that aren't related to my PR.

[jorgemoya]

pnpm-lock shouldn't need to be updated. Component changes look good tho!

@jorgemoya I assume it changed the lock file by me just running pnpm install. Do I just need to undo that change in the file and push it back up?

I also was about to ask in Slack what I should do about the lint failures in the build that aren't related to my PR.

I would just undo the file and push it back up. We can then investigate if we need to update the lock file.