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 sign #548

Merged
merged 21 commits into from
Apr 2, 2024
Merged

Add sign #548

merged 21 commits into from
Apr 2, 2024

Conversation

BenCheung0422
Copy link
Member

@BenCheung0422 BenCheung0422 commented Aug 2, 2023

Resolves #351
The original pull request doing this is #366 and has been closed because of the incompatibility of the tile handling system.
Currently, there is a way do that by adding a separated save file for signs, but it still needs to be combined into a standardized system of tile data save system in the future. Signs could make more contents within structures when complementing the storyline. As mentioned in #542, tile entities could be made with signs.

@BenCheung0422 BenCheung0422 marked this pull request as ready for review August 3, 2023 15:15
Copy link

@phithagod24 phithagod24 left a comment

Choose a reason for hiding this comment

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

Nice!

@Litorom
Copy link
Member

Litorom commented Jan 25, 2024

Could you please resolve conflicts. I believe this can work with storyline.

# Conflicts:
#	src/client/java/minicraft/core/io/InputHandler.java
#	src/client/java/minicraft/item/Items.java
#	src/client/java/minicraft/level/Level.java
#	src/client/java/minicraft/level/tile/Tiles.java
#	src/client/java/minicraft/level/tile/TorchTile.java
@BenCheung0422
Copy link
Member Author

Conflicts are now resolved.

Copy link
Member

@Litorom Litorom left a comment

Choose a reason for hiding this comment

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

Sign display still shows sometimes after leaving a sign tile.

@BenCheung0422
Copy link
Member Author

It should be fixed now.

Copy link
Member

@Litorom Litorom left a comment

Choose a reason for hiding this comment

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

Upon editing the sign in creative, you can use Debug cmds and switch levels when in sign edit menu

@BenCheung0422
Copy link
Member Author

Okay, actually that happens to every menu, even pause menu.

@BenCheung0422
Copy link
Member Author

In this case, #504 should have already avoided this from happening.

Copy link
Member

@Litorom Litorom left a comment

Choose a reason for hiding this comment

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

This is not what I meant! Please change it back and add menu checks

@BenCheung0422
Copy link
Member Author

Okay I added it back and moved to another place.

@BenCheung0422
Copy link
Member Author

The code will be updated based on #555.

Litorom and others added 4 commits March 19, 2024 15:29
# Conflicts:
#	src/client/java/minicraft/level/Level.java
#	src/client/java/minicraft/level/tile/Tiles.java
#	src/client/java/minicraft/level/tile/TorchTile.java
#	src/client/java/minicraft/saveload/Load.java
@Litorom Litorom merged commit c7a2a60 into MinicraftPlus:main Apr 2, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

Add signs
4 participants