Add support for back_patch, front_patch to loader entity (from underground belt)

Post Reply
User avatar
kirazy
Inserter
Inserter
Posts: 29
Joined: Tue Mar 06, 2018 12:18 am
Contact:

Add support for back_patch, front_patch to loader entity (from underground belt)

Post by kirazy » Fri Mar 01, 2019 5:53 am

Image

The underground belt in 0.17 has a front_patch and back_patch property that helps with the layering around items and other structures.

I'd like to request this property be applied to the loader as well, so that I can make 0.17-vanilla-style sprites as well. Currently it's not possible. :[

Image

User avatar
Deadlock989
Smart Inserter
Smart Inserter
Posts: 1074
Joined: Fri Nov 06, 2015 7:41 pm
Contact:

Add underground belt back_patch & front_patch support to loaders

Post by Deadlock989 » Mon Mar 18, 2019 8:26 pm

Hi,

With 0.17 and the graphics reworking, underground belts got a new back_patch and front_patch component definition in the sprite structure:

Code: Select all

["underground-belt"] = {
    ["express-underground-belt"] = {
      ...
      structure = {
        back_patch = {
        ...
        },
        direction_in = {
        ...
        },
This lets you simulate the appearance of a 3D structure to the underground, letting a piece of the structure get drawn behind the belt feeding into the underground.

Loaders followed the same sprite structure before 0.17, and they are similar to undergrounds in many other ways (belts feeding in and out, rotatable between in/out belt direction instead of turning around, same sprite definitions etc.). However in tests with a new model for loaders, it seems the back_patch component of the sprite structure is ignored - see pic below.

Could these be added to loaders as well please? Many thanks.

Image

Boodals
Long Handed Inserter
Long Handed Inserter
Posts: 93
Joined: Sun Feb 11, 2018 7:10 pm
Contact:

Re: Add underground belt back_patch & front_patch support to loaders

Post by Boodals » Tue Mar 19, 2019 8:11 pm

This has already been requested: 66071

Edit by Bilka: Merged with the above topic

User avatar
kirazy
Inserter
Inserter
Posts: 29
Joined: Tue Mar 06, 2018 12:18 am
Contact:

Add "back_patch", "front_patch" structure properties to Loader prototype (from Underground Belt prototype)

Post by kirazy » Sun Jun 02, 2019 8:03 am

The underground belt prototype has optional structure properties "back_patch" and "front_patch".

I request that these two optional properties be added to the loader prototype as well, so that we can replicate the graphical style properly with the new belts.

Image

Currently without these properties the texture sits on top of the belt (and items on the belt!) graphics when using the vanilla textures.

Any hope of this? :C

User avatar
Deadlock989
Smart Inserter
Smart Inserter
Posts: 1074
Joined: Fri Nov 06, 2015 7:41 pm
Contact:

Re: Add "back_patch", "front_patch" structure properties to Loader prototype (from Underground Belt prototype)

Post by Deadlock989 » Sun Jun 02, 2019 9:18 am

I asked about this before and was told it won't be sorted out until they know what to do with the vanilla loaders in the campaign.

User avatar
kirazy
Inserter
Inserter
Posts: 29
Joined: Tue Mar 06, 2018 12:18 am
Contact:

Re: Add "back_patch", "front_patch" structure properties to Loader prototype (from Underground Belt prototype)

Post by kirazy » Sun Jun 02, 2019 9:21 am

Hm. Interesting. Thanks for the info, though!

User avatar
Deadlock989
Smart Inserter
Smart Inserter
Posts: 1074
Joined: Fri Nov 06, 2015 7:41 pm
Contact:

Re: Add "back_patch", "front_patch" structure properties to Loader prototype (from Underground Belt prototype)

Post by Deadlock989 » Sun Jun 02, 2019 9:27 am

Just in case it looked like I was saying otherwise - this request gets a massive +1 from me. Functionally, loaders are very similar to undergrounds and it makes complete sense that they'd have the same sprite structure.

Bilka
Factorio Staff
Factorio Staff
Posts: 2030
Joined: Sat Aug 13, 2016 9:20 am
Contact:

Re: Add support for back_patch, front_patch to loader entity (from underground belt)

Post by Bilka » Sun Jun 02, 2019 9:37 am

Hey, I merged 3 different topics that asked for this. Please don't make new topics when there are already existing topics asking for exactly the same thing. It makes it harder for us to keep an overview of who wants what and why. Thank you.
I'm an admin over at https://wiki.factorio.com. Feel free to contact me if there's anything wrong (or right) with it.

User avatar
Raiguard
Long Handed Inserter
Long Handed Inserter
Posts: 75
Joined: Wed Dec 13, 2017 8:29 pm
Contact:

Re: Add support for back_patch, front_patch to loader entity (from underground belt)

Post by Raiguard » Tue Aug 06, 2019 5:33 pm

Is there a chance of this happening now? It seems that using loaders in the campaign has been scrapped.

My "Infinity Loader" entity uses an underground belt because it has front_patch and back_patch support. If loaders supported this, then I could use that instead and avoid a bunch of issues relating to using underground belts with a connection distance of zero (since this entity spawns its own items, it doesn't connect to another underground belt).
2019-08-06 11_32_11-Window.png
2019-08-06 11_32_11-Window.png (165.52 KiB) Viewed 779 times
Don't forget, you're here forever.

Bilka
Factorio Staff
Factorio Staff
Posts: 2030
Joined: Sat Aug 13, 2016 9:20 am
Contact:

Re: Add support for back_patch, front_patch to loader entity (from underground belt)

Post by Bilka » Wed Aug 14, 2019 11:58 am

Staplergun added this in 0.17.65.
I'm an admin over at https://wiki.factorio.com. Feel free to contact me if there's anything wrong (or right) with it.

Post Reply

Return to “Implemented mod requests”

Who is online

Users browsing this forum: No registered users