Chad Lee
Chad Lee
Yes, that seems like a good implementation. I would be appreciative if you wanted to submit a PR.
There aren't currently options to do that, but it should be relatively straight-forward to update the CSS and add an option for that. I'd be happy to review a PR...
Thanks for this! I'm wondering if it makes sense to just do a major version bump and include react-icons as a normal dependency.
https://twitter.com/cassidoo/status/1255690494077628418 I'm in the "never, never" stage of typescript right now.
We implement it using jss so that the styles can be overridden. To fix your issue, we would need to add `secondary`, `light`, & `dark` to [`styles.js`](https://github.com/chadly/react-bs-notifier/blob/master/src/alert/styles.js). I'd be happy...
ohhh neato, I didn't know that existed. I'd be happy to take a PR
Why do you not want JSS to appear in the bundle? Is it the bytes you are worried about? How much space did you save replacing JSS with LESS? TBH,...
Please don't do this. Strong naming is the bane of my existence. I'd rather you just recommend [Strong Namer](https://github.com/dsplaisted/strongnamer) for the people that really want it.
I'm very much a typescript believer now. If someone wants to use ChatGPT to convert this all to TS, I'd be happy to review the PR.