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

Gunpowder #14

Merged
merged 9 commits into from Dec 24, 2022
Merged

Gunpowder #14

merged 9 commits into from Dec 24, 2022

Conversation

Dream-Master
Copy link
Member

Gunpowder recipe remove
remove tooltip

@Dream-Master Dream-Master requested review from a team December 24, 2022 13:25
Copy link
Member

@glowredman glowredman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The entries in the lang files need to stay too.

@@ -1110,9 +1110,6 @@ public void addRecipes() {
// Clouds
GameRegistry.addRecipe(stackSingleCharcoal, "ccc", "ccc", "ccc", 'c', new ItemStack(cloud, 1, 2));
GameRegistry.addRecipe(new ItemStack(plantItem, 1, 4), "cc", "cc", 'c', new ItemStack(cloud, 1, 3));
GameRegistry.addRecipe(
new ShapedOreRecipe(new ItemStack(Items.gunpowder, 1, 0), "cc", "cc", 'c', "dustSulfur"));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't remove the recipe. Just put it in a if (!Loader.isModLoaded("dreamcraft")) { } block instead.

@@ -65,15 +65,12 @@ public void addInformation(ItemStack stack, EntityPlayer player, List list, bool
list.add(StatCollector.translateToLocal("tooltip.cotton"));
break;
case 4:
list.add(StatCollector.translateToLocal("tooltip.sulfur"));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

list.add(StatCollector.translateToLocal("tooltip.fletching"));
break;
case 6:
case 5:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just because you removed a case doesn't mean these need to be changed, this will cause wrong tooltips on the other items

list.add(StatCollector.translateToLocal("tooltip.imp"));
break;
case 7:
case 6:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See above

@github-actions
Copy link

Warning: 2 uncommitted changes
#15

Co-authored-by: Martin Robertz <dream-master@gmx.net>
Co-authored-by: GitHub GTNH Actions <>
@Dream-Master Dream-Master merged commit 6b3a714 into master Dec 24, 2022
@Dream-Master Dream-Master deleted the Gunpowder branch December 24, 2022 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants