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

[WIP] Add tarped wagon to support unidentified vehicles #4

Merged
merged 1 commit into from
Jul 8, 2017

Conversation

brantwedel
Copy link
Contributor

@brantwedel brantwedel commented Jul 7, 2017

Made a comment, took some action, my first stab at working with factorio mod system: https://mods.factorio.com/mods/Supercheese/Vehicle%20Wagon/discussion/6587

This allows any vehicle to be loaded onto the wagon, if the vehicle is not detected, it falls back to a tarp covered wagon graphic.

This is [WIP] because I think the tarp graphic could be greatly improved (It's basically a modified tent model I found https://free3d.com/3d-model/puo-66928.html), I'm thinking of doing something similar to this if I have time this weekend: http://www.modelcrafters.com/wordpress/?product=us-army-53-war-emergency-gondola-wsplit-tarp-covered-load

screenshot 2017-07-06 23 35 49

Also, I kind of guessed on the small German localization change

I experimented with inserting items to show what was under the tarp when in "alt-mode" for when you have too many tarped wagons but that code isn't included because it was too hacky ... wonder if you have any ideas for showing that ...
screenshot 2017-07-06 22 29 06

@Suprcheese Suprcheese merged commit 7bfd681 into Suprcheese:master Jul 8, 2017
@Suprcheese
Copy link
Owner

Ooh, I quite like this! I'll start testing immediately.

@Suprcheese
Copy link
Owner

This is a splendid addition indeed, and I've added it in v1.1.2.
Thanks so much for being passionate enough to take the time to add this improvement!

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