-
Notifications
You must be signed in to change notification settings - Fork 808
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
[added] add class to html when modal is open #588
Conversation
This solves my problem from #585 completely |
@bannier Thanks!! I'll review your PR later. Looks good. |
https://github.com/reactjs/react-modal/blob/master/examples/basic/app.css This is the example of css for the tests, can you include the class for the |
I don't know if we should use <Modal htmlOpenClassName={string || null} ... /> If Any thoughts? |
cc @bannier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting for feedback.
sorry I haven't checked my PR since last week. Sure I can add an option for that. But 2 questions still :
|
I don't know,...It would keep the same behavior for old applications and only enable this for projects that will need this. We will let the same behavior for Hope this make sense. |
It makes sense. I'll do it. |
Let me know if you need any help, @bannier? |
Uhm.. why not merging this?? I have the same issue.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes are mentioned in the comment section.
Sorry @diasbruno I had issues with adapting tests to this option and didn't took a second shot at it since. Work in progress here |
No problem, @bannier. Please, let me know if you need anything. |
Nice job! |
Guys, more than 1 month has gone. Would it be finally merged or not? |
overflow:hidden
onbody
is not enoughoverflow:hidden
onbody
is not enoughChanges proposed:
html
when modal is openAcceptance Checklist:
CONTRIBUTING.md
.This addition involves renaming some helpers from
bodyClass
to more generic terms.