-
Notifications
You must be signed in to change notification settings - Fork 36
Description
First off I want to say that this is awesome. Thanks for putting this together for us.
One suggestion though is to remove excess styling that we do not need. It is just a pain to have to over write all these css styles for things that Im assuming most people are going to change. So far I have run into background color, font-size in the step class and backgrounds and padding on the a tag in the step.
I think it would be better if this was setup with out any styles that it did not need to do the animations to move from step to step. That way it would inherit my pre existing css and I would not have to even think about fighting with the styles that are there already.
Another thing to think about is to allow passing inline styles into the components. A lot of react projects have tried to move away from the idea of global css styles and it would be nice to not need to define classes just for this project.