-
-
Notifications
You must be signed in to change notification settings - Fork 111
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
Added dwarfFortress
spinner
#65
Conversation
@@ -40,6 +40,7 @@ | |||
"json" | |||
], | |||
"devDependencies": { | |||
"@types/node": "^17.0.41", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re: this change, it looks like there was a breaking change to this package - see this stackoverflow post. Happy to adopt another approach if you want.
Also - happy to uprev the min version to node 8 if you're so inclined for deprecation/unit test reasons, but didn't want to do it unilaterally.
spinners.json
Outdated
@@ -1464,5 +1464,143 @@ | |||
"▰▰▰▰▰▰▰", | |||
"▰▱▱▱▱▱▱" | |||
] | |||
}, | |||
"dwarfFortress": { | |||
"interval": 100, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The animation feels a bit slow. 80 seems a bit smoother.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, makes sense.
This is a fun one 👍 |
This adds an extremely silly spinner for when you've progressed too far and unleashed something extremely Fun. Based off of Dwarf Fortress's ASCII tileset.