-
Notifications
You must be signed in to change notification settings - Fork 33
Support themes from mdThemingProvider #6
Comments
Sure, go for it. I haven't been able to really look into angular material, though it looks awesome, as my use has been in bootstrap so I have no real knowledge about the themeing included or time to figure it out at the moment. Change as much or as little as you need, I don't really mind as long as it still works without the material stuff - I'm happy to check through this once you've done the PR. |
I quite like the idea of splitting up the style. That way, for non angular material users, I can update the instructions to include it, much like you have to include a shim for md-button. Will send you a PR when ready! |
+1 |
Any updates on this as I too need to use the control on my website and have it follow the theme using themeprovider. |
+1 |
1 similar comment
+1 |
I've done some work now with angular-material, and there's a big issue with trying to get into this stuff in nicely as pointed out here: angular/material#1269 (<- this is one of the reasons we gave up on angular-material too...) Until that issue is solved I think the best way is to generate your own stylesheets from the stylus (less and sass version would look 90% the same - I may upload a sass option at some point) as we don't know what the "Official" solution will be sadly. |
Moving to new repo sc-date-time/sc-date-time#1 |
We are using this component in a site powered by angular material and I was looking to enable mdTheming support so that it blends in with the rest of the site.
This should be as simple as updating the html to use
md-classnames
.We could make this an option passed to the jade to allow use without angular material.
Was wondering if you would be open to this as a PR.
The text was updated successfully, but these errors were encountered: