vmware-clarity / core

Clarity is a scalable, accessible, customizable, open-source design system built with web components. Works with any JavaScript framework, created for enterprises, and designed to be inclusive.
https://clarity.design
MIT License
163 stars 42 forks source link

fix(modal): add bottom margin without cds-modal-actions #175

Closed ashleyryan closed 1 year ago

ashleyryan commented 1 year ago

fixes #174

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

What is the current behavior?

If you don't have a cds-modal-actions element, there is no margin at the bottom of the modal. So you need the element even if you don't have any actions, which actually provides more margin than necessary due to the gap CSS between elements.

Issue Number: #174

What is the new behavior?

Padding is applied to the container, so will appear regardless of the elements rendered

Does this PR introduce a breaking change?

Other information

github-actions[bot] commented 1 year ago

๐Ÿ‘‹ @ashleyryan,

Thank you,

๐Ÿค– Clarity Release Bot

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 6.1.6 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

github-actions[bot] commented 1 year ago

Hi there ๐Ÿ‘‹, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.