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

Allow breaking cobwebs as fast as swords #4

Closed
p3lim opened this issue Jul 10, 2022 · 4 comments
Closed

Allow breaking cobwebs as fast as swords #4

p3lim opened this issue Jul 10, 2022 · 4 comments

Comments

@p3lim
Copy link

p3lim commented Jul 10, 2022

No description provided.

@Fourmisain
Copy link
Owner

Personally I think it's fine that they don't, since that factors into the choice between Sword and Axe and you can always use Shears.
So I'd make this into an option, off by default.

But some things are not quite clear:
Should the string drop as well?
Should this also work with other tools when "All Tools Are Weapons" is enabled?

@Fourmisain
Copy link
Owner

k, I think the answer to both of those questions is yes.

Breaking Cobwebs fast but not having them drop String feels weird because blocks being broken fast communicates you're using the proper tool, hence it should also drop the String.

While it is kinda weird mining Cobwebs with a Pickaxe, Shovel or Hoe, the same can be said about them being used as weapons and since all other options of this mod apply to them equally, so should this option for consistency.

I got working versions for Fabric 1.16 and 1.17 now, the latter which seems to work fine on 1.18 and 1.19 too.
Forge needs some testing and I need to get that publishing script going (because I need to stop manually releasing 5+ builds for each small change onto multiple platforms).
Hopefully tomorrow.

@Fourmisain
Copy link
Owner

New versions are up!

Aaand I just noticed this doesn't work server-only.
Singleplayer is fine but for multiplayer the clients need to have the mod installed with the setting enabled, else it'll break slowly.

@Fourmisain
Copy link
Owner

Moved this new issue to #5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants