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(modal): add modalDialogClass to modal-dialog div #3826

Open
wants to merge 1 commit into
base: master
from

Conversation

@MisuleT
Copy link

@MisuleT MisuleT commented Aug 5, 2020

Before submitting a pull request, please make sure you have at least performed the following:

  • read and followed the CONTRIBUTING.md and DEVELOPER.md guide.
  • built and tested the changes locally.
  • added/updated any applicable tests.
  • added/updated any applicable API documentation.
  • added/updated any applicable demos.
@jnizet
Copy link
Collaborator

@jnizet jnizet commented Aug 5, 2020

Hi @MisuleT

Thanks for the PR, but could you please explain why you think we should add such a configuration option? Can't you use the already supported windowClass option?

For example with such a selector in your stylesheet:

.myCustomWindowClass .modal-dialog {
  ...
}
@MisuleT
Copy link
Author

@MisuleT MisuleT commented Aug 6, 2020

Hi @jnizet

We depend on another library in our project, that is based on Bootstrap. We already use windowClass, but we have to add another class to modal-dialog div for the correct library function.

We created workaround where we add class 'g-modal' to size option. But it is not clear solution and if you change this option it won't work for us.

options.size += ' g-modal';

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.