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

Added Paintbrush GUI and Dyamic Overlay System #682

Conversation

02skaplan
Copy link

  • Added paintbrush GUI that displays textures on a rotating "carousel."
  • Added texture description system to display information about a model and or texture in the paintbrush GUI.
    • Example on Forney.
  • Ported overlay texture system from RWCTC for writing text on textures, destination boards, and the like.
    • Example on BAP ES44 (Sorry, Bida).

Recommended to squash commits if/when merged.

TC4.5.Paintbrush.Demo.mp4

- Paintbrush ported in as-is from RWCTC before modification.
- Unable to perform meaningful testing with texture system out of commission.
- Rearranging paintbrush menu & buttons to more closely resemble desired layout.
…unity-4.3

# Conflicts:
#	src/main/java/train/client/render/RenderRollingStock.java
#	src/main/java/train/common/api/AbstractTrains.java
#	src/main/java/train/common/api/EntityRollingStock.java
- Paintbrush menu almost in finished state.
- Reworked tooltip calls to increase performance significantly.
- Paintbrush completed per design specifications.
…unity-4.3

# Conflicts:
#	src/main/java/train/client/render/RenderRollingStock.java
@02skaplan
Copy link
Author

Forgot to add the video demonstrating the dynamic overlay system. GUI layout & design can be changed at a later date if necessary, but no mention of this was made to me, so I ported it in as-is. Localization for the overlay controller buttons in the main paintbrush menu is also missing — I will add this in a later commit.

TC4.5.Dynamic.Overlay.System.Demo.mp4

@EternalBlueFlame EternalBlueFlame merged commit fe53858 into EternalBlueFlame:1.7-Community-4.3 Nov 28, 2023
1 check 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.

2 participants