-
-
Notifications
You must be signed in to change notification settings - Fork 169
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
Add Item Abilities for lighting and dowsing #1219
Conversation
|
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 dowsing item ability seems fine to me, however the lighting item ability didn't got fire charge included.
Note that flint and steel may also be used to light blocks using dispenser, a patch to DispenseItemBehavior
is also needed for consistency.
Also renamed the ability to be more generic, now that it is used by two items.
Wouldn't the proper term here be "dousing"? At the very least, that would be the more common spelling, and it would risk less confusion with the divination practice of dowsing. |
The vanilla name is |
hmm if we are going to have confusion over the word, is there an alternative word we can use for unlighting? |
how about extinguish^^? That's what I would have went with anyway before we have a 1984 moment (and actually call it unlighting^^) ... |
While there's some precedence (mc has "Candle extinguishes" as subtitle), I would prefer to not use words people generally use when talking about FireBlock fires when talking about actions on blocks that can be "turned on and off". |
Implements #1179 by adding an ItemAbility for Flint&Steel to light candles, cake with candles and campfires. Also adds an ItemAbility for the reverse, i.e. for shovels to dowse campfires.
Edit: And FireCharge and dispenser using Flint&Steel.