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 half panel display #16

Merged
merged 2 commits into from
May 21, 2024
Merged

Add half panel display #16

merged 2 commits into from
May 21, 2024

Conversation

slmnio
Copy link

@slmnio slmnio commented May 18, 2024

Hi there! I really love this mod and I'm planning to use it with our upcoming server.

After playing with in our existing worlds, I wanted to make a version of the small display but with the thickness of the panel display, since space is usually limited inside trains. Whilst I am a developer, I don't have much experience with Minecraft mods. I believe I have set everything up correctly, using the existing code for the small and panel displays.

image

I worked on this inside IntelliJ IDEA and have it working through the fabric client through there, testing all of the orientations.
However, I am unable to build and create a jar file due to some errors such as Cannot nest jars into none mod jar createrailwaysnavigator-0.5.2-beta-1.20.1-common.jar. I don't know enough about Minecraft modding to be able to sort this out. If you have any tips, or if you could build it and upload the jar it to this PR, that would be awesome.

This display feels complete with the rest of the displays in the mod, especially since having the smaller font size and the 2:1 aspect ratio means you see more information. Of course, you have no obligation to include this into the mod, but I thought I might as well offer up the code back to you. Thank you for the time and effort you've put into the mod already.

@MisterJulsen MisterJulsen added the enhancement New feature or request label May 19, 2024
@MisterJulsen MisterJulsen marked this pull request as ready for review May 21, 2024 16:02
@MisterJulsen
Copy link
Owner

Thank you for your contribution! Unfortunately, I don't know about this error and therefore can't say exactly why it doesn't work without further details. But I am very happy to integrate your new display variant into the mod. 😁

@MisterJulsen MisterJulsen merged commit 89c53c9 into MisterJulsen:1.20.1 May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants