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
Support multiple classes in MdDialog's panelClass #6318
Comments
Should be able to do
That said, I wish all the overlay class api's were NgClass-compatible |
@willshowell I believe that is not possible, doing
gives me next error:
etc. |
Oh yeah, you're right. Maybe some combination of this with an - this._pane.classList.add(this._state.panelClass);
+ this._pane.classList.add(...this._state.panelClass); |
Allows for more than one class to be added via the `panelClass` option. Fixes angular#6318.
Allows for more than one class to be added via the `panelClass` option. Fixes #6318.
Allows for more than one class to be added via the `panelClass` option. Fixes #6318.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug, feature request, or proposal:
Feature request
What is the expected behavior?
I want MdDialogConfig to have option for panelClass to be an array of classes, like
panelClass: ['custom-container', 'widget-container']
What is the current behavior?
Currently there could be only one panelClass per dialog
What is the use-case or motivation for changing an existing behavior?
I want to add various classes depending on environment, but i have no such option and i have to include all possible combinations into separate classes
The text was updated successfully, but these errors were encountered: