-
Notifications
You must be signed in to change notification settings - Fork 266
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
Change Nanobar Color after creating it #35
base: master
Are you sure you want to change the base?
Conversation
Could you merge this? I need to change the default color of my nanobar. Thanks! |
Looks like the author of this package is no longer actively maintaining it :-(
|
That's the fate of so many (open source projects). Sooner or later, creators get bored and move on to other things. Because of this, I have recently realized that forks are your friends. If an original source creator is no longer around, fork his branch, manage it as your own and don't forget to share (let the world know). Better pick the project carefully though; some code bases are more easily maintained than others. Legibility matters. |
I've given some thought to your changes BTW. I'm not sure that styling in code is such a good idea (which is what setColor does). I think the simple addition of a CSS class attribute (e.g., "bar") to the inner div would be better. That way, you've just (and simply) given a way for the "stylists" to do their work the standard way. Food for thought. |
Hey, yea, unfortunately forks can be your best friend at times. My changes were just a quick fix because I needed a fast way to modify the colors, maybe we could switch over to my fork and discuss how we could implement a better way to modify styles? |
Sorry guys, I do not look often notifications. Thanks for the work @fearhat , but I made a new nanobar. I plan to release it in a few days, you can take a look at it here: https://github.com/jacoborus/nanobar/tree/new |
Hey! Wow, that's even better! Thanks! I'll check it out asap. :-) |
see commits (only done on index.js, should be easy on the other ones though)