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

Tweak Excite Truck Textures for consistency #199

Merged
merged 1 commit into from
Aug 1, 2023
Merged

Tweak Excite Truck Textures for consistency #199

merged 1 commit into from
Aug 1, 2023

Conversation

teconmoon
Copy link
Contributor

@teconmoon teconmoon commented Jul 31, 2023

The Excite Truck textures being generated using the text-outlines style looked a bit odd, and some of the controller textures seen in the training mode were overlapping. This PR is intended to address those issues.

  • Tweaked the GameID allowing compatibility with other regions
  • Moved all texture coordinates to latest Defaultstyle JSON template
  • Tweaked coordinates to improve display of generated "tilt" textures and avoid weird masking issues
  • Added device textures for training mode
  • Cleaned up unnecessary texture files

Copy link
Owner

@Venomalia Venomalia left a comment

Choose a reason for hiding this comment

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

The changes to the devices must be reversed.

Excite Truck/UniversalDIT_Defaultstyle.json Outdated Show resolved Hide resolved
@teconmoon
Copy link
Contributor Author

@Venomalia Sorry can you check it again? I tried re-copying the devices section from UniversalDIT_Defaultstyle.json in #Assets and pasting it over the Excite Truck JSON. I'm not really sure how it got changed. Sorry for the trouble.

@Venomalia
Copy link
Owner

there is the possibility to squash several commits into one which makes it clear to view for next time.

I simply did that now.

@Venomalia Venomalia merged commit a812d1c into Venomalia:main Aug 1, 2023
@teconmoon teconmoon deleted the Excite-Truck branch August 4, 2023 14:52
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