Skip to content
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

Merged
merged 6 commits into from
Jul 14, 2024

Conversation

HenryLoenwind
Copy link
Contributor

@HenryLoenwind HenryLoenwind commented Jul 3, 2024

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.

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

Copy link
Contributor

@RaymondBlaze RaymondBlaze left a 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.
@Matyrobbrt Matyrobbrt added enhancement New (or improvement to existing) feature or request 1.21 Targeted at Minecraft 1.21 labels Jul 4, 2024
@sciwhiz12
Copy link
Member

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.

@HenryLoenwind
Copy link
Contributor Author

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 CampfireBlock.dowse(); I kept that for consistency. (And because I had no reason to look up the spelling. Neither dowse nor douse are words I do anything with but recognise them when I read them...)

@TelepathicGrunt
Copy link
Contributor

image
Sometime mojang typos lol. We should do correct spelling imo

@HenryLoenwind
Copy link
Contributor Author

HenryLoenwind commented Jul 7, 2024

While I don't think the definition is that clear, with even the Uk parliament using dowsing and some dictionaries listing it as variant spelling, I have no attachment to that name and will change it.

image

PS: For reference (i.e. to save myself from having to search again and again):

image
image

@TelepathicGrunt
Copy link
Contributor

hmm if we are going to have confusion over the word, is there an alternative word we can use for unlighting?

@Ezelsorix
Copy link

Ezelsorix commented Jul 8, 2024

how about extinguish^^? That's what I would have went with anyway before we have a 1984 moment (and actually call it unlighting^^) ...

@HenryLoenwind
Copy link
Contributor Author

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".

@TelepathicGrunt TelepathicGrunt merged commit 3c03524 into neoforged:1.21.x Jul 14, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.21 Targeted at Minecraft 1.21 enhancement New (or improvement to existing) feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants