mspielberg / factorio-miniloader

UPS-friendly 1x1 psuedo-loaders for Factorio
GNU Lesser General Public License v3.0
24 stars 21 forks source link

Graphics update! #40

Closed kirazy closed 4 years ago

kirazy commented 4 years ago

I made these masks for Krastorio... but I thought rather than try to make a mod to reskin Miniloaders it would be so much easier to just... update Miniloaders.

Specific changes:

The core miniloaders: image

Ultimate belts, as an example, but all the supported mods have updated color matching: image

As an aside, I think you can probably drop support for Xander's mod and the old Krastorio mod. In Xander's case, there's no more expedited belt that I could find. And the old Krastorio mod is broken by a recent vanilla update (and is deprecated so will never work again).

kirazy commented 4 years ago

Oh, forgot to include pictures of in-game icons/tech icons: image image

kirazy commented 4 years ago

I'm happy to make the changes myself. Also thanks for bringing util.color to my attention, now I have to go make some changes after this in another of my mods...

kirazy commented 4 years ago

Finished make the changes you asked for.

bitwiseshiftleft commented 4 years ago

Thanks for all your work on this!

I do have a concern about the new graphics, though. In my opinion the filtered and unfiltered miniloaders look too similar, so that it's hard to tell them apart at a glance either in inventory or once placed. (Edit: This was a problem in the previous graphics on the red loaders, but not as much on other colors.) Also, vertically oriented miniloaders and underneathies are very similar, despite doing quite different things.

So maybe next time you change the graphics, could you consider making them more visually distinct? Like maybe they could have some kind of controller on top, that would glow or otherwise be a different color on filtered miniloaders, instead of just a pink tint to the metal?