Skip to content

Conversation

@Wawrzyn321
Copy link
Contributor

Description

  • Improve Modal by allowing the user to add CSS classes to modal body. Previously, it was possible to add the class only to the backdrop. It could be worked around as described here.
  • I've changed the current prop name from className to backdropClassName to avoid confusion.

fixes #800

@claassistantio
Copy link

claassistantio commented Dec 5, 2019

CLA assistant check
All committers have signed the CLA.

@claassistantio
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Wawrzyn321 Wawrzyn321 changed the title fix: allow adding classes to both modal body and backdrop feat: allow adding classes to both modal body and backdrop Dec 5, 2019
@netlify
Copy link

netlify bot commented Dec 5, 2019

Copy link
Contributor

@jbadan jbadan left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! 🚢

One of the team will merge for you once the CLA bot approves. Already signed

@jbadan jbadan merged commit fd52cf0 into SAP:master Dec 5, 2019
jacobdevera added a commit that referenced this pull request Dec 17, 2019
* fix: use momentjs

* fix: next/prev handling

* feat: allow adding classes to both modal body and backdrop (#816)

BREAKING CHANGE:
* removed className prop, new className props

* chore(release): version 1.0.0-rc.0 build 3065 [ci skip]

* fix: use moment in datepicker

* fix: simplify derived state logic

* fix: simplify enabled date logic

* fix: simplify validation logic

* fix: tests

* fix: localization

* fix: increase size limit

* fix: use date objects for props

* chore: calendar stories

* chore: datepicker stories

* chore: locale

* merge

* fix: unnecessary diffs

* fix: visual regression testing
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.

Modal does not honor provided CSS class in "className" property

3 participants