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

Further work since last request, diff now displaying correctly. #314

Merged
merged 6 commits into from
Jan 16, 2022
Merged

Further work since last request, diff now displaying correctly. #314

merged 6 commits into from
Jan 16, 2022

Conversation

itsjustcurtis
Copy link
Contributor

Previous commits are still shown on history, most recent two commits are the relevant ones.

Added functionality to account for Addon Paint Colours, found in "Extra Colours" menu item. Menu option is hidden if no colours are found.
Changed Index Selector to dynamically change size based on number of added Colours, rather than capped at 165.
Changed Cyan RGB and Trippy Green RGB to Random Index and Random RGB
Paints section overhauled to allow selection of added colours in carmodcols. Credit to sjaak for basic functionality and assistance.

Colour indexing no longer limited to 165, number selection changes in size dynamically depending on the number of new colours.

Main colour selections (Classic, Metallic, Matte etc...) are now dynamic lists based on in-game carmodcols, not hard-coded.

Cyan and TrippyGreen RGB selections removed in favour of Random RGB and Random Index options.

Addition of Drift Tyres Toggle

Addition of Lower Suspension Toggle (only works on Tuners update vehicles, toggle is persistent and does not reset with a new vehicle.)
@MAFINS MAFINS merged commit 4ee10e8 into MAFINS:master Jan 16, 2022
@itsjustcurtis
Copy link
Contributor Author

@MAFINS appreciate the merge, I've just realised I may not have included the release .asi file. Do you need me to do a new pull request for that to be included in a new release version 1.6.0?

@MAFINS
Copy link
Owner

MAFINS commented Jan 17, 2022

@itsjustcurtis Do not worry about that. The build is automated (appveyor).
In fact, the Menyoo.vcxproj should've also been ignored since certain version numbers were changed that didn't need to be included and made the build stop working. I subsequently restored those version numbers.

@itsjustcurtis
Copy link
Contributor Author

@MAFINS ahh okay, only reason I ask is that I tested the release version and the changes didn't seem to migrate across. I recently released a colour pack for the game and intend to direct people to the latest version. I just know that if it doesn't display the colours I'll be inundated with messages saying it doesn't work!

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