-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Give Drill type drawers #10524
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
base: master
Are you sure you want to change the base?
Give Drill type drawers #10524
Conversation
can you show the difference side by side? |
No visual differences just more mod customization |
alr, so i don't see anything wrong here |
No legacy drawRim support. Will cause issues with every mod using rims. |
As I said in the discord I'm looking for ways |
just make DrawRim do nothing if no rim sprite is found |
//does nothing for most Drills, as those do not require items. | ||
if((consTimer += delta()) >= consumeTime){ | ||
consume(); | ||
consTimer %= 1f; |
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.
Shouldn't it be %= consumeTime
?
|
||
return (drillProgress + consumeProgress) / 2f; |
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.
Why is consumeProgress even a factor here? It has nothing to do with progress until the next item. Why is it averaged?
|
||
return (drillProgress + consumeProgress) / 2f; | ||
} | ||
@Override |
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.
weird indent
Fixing issues later today |
If your pull request is not translation or serverlist-related, read the list of requirements below and check each box:
Gives drawers to drills whilst preserving most base drill stuffs.