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

support for RF concrete #30

Open
FluffierThanThou opened this issue Sep 30, 2020 · 3 comments
Open

support for RF concrete #30

FluffierThanThou opened this issue Sep 30, 2020 · 3 comments

Comments

@FluffierThanThou
Copy link
Member

Reported by: Heart of Mud
Description:
Hi Fluffy, about RF's Concrete, I don't know if it's me and my mods being in the wrong order that's at fault or if it's just not supported, but the brick floor and herringbone brick floor of that mod isn't replaced by the equivalents from this mod. Also, the raw material 'bricks' from that mod can be used to make 'brick brick' floors from your mod, so it's a bit messy. I thought that concrete mod was widely used so was surprised to see it appear to not be supported. Here it is for reference: https://steamcommunity.com/sharedfiles/filedetails/?id=2012397587

@FluffierThanThou
Copy link
Member Author

Heart of Mud 4 hours ago
I'm sorry for misinforming you, but I realised Fertile Fields is the mod that provides the brick stone type and the 2 duplicate brick floor types.
https://steamcommunity.com/sharedfiles/filedetails/?id=2012735237
That's my bad for not double checking before posting!

@InsanityPrelude
Copy link
Contributor

Fertile Fields brick floors (like the walls) have separate gizmos and such because it's the only way to preserve the white/grey gradient of the mortar in between the bricks. If anything, bricks should not be registered as a "rocky material" because it means they're registered twice everywhere. As well, with stuffed floors you can make weird things like "brick flagstone" and "brick bricks." A "brick" is a style or cut of stone, not a material in and of itself.

A patch could perhaps rename the bricks to "Clay-fired bricks" or "Shale bricks" or something, but you'd still have at least one entry that's just "Shale brick bricks" because one of the floor types in Stuffed Floors is just called "bricks."

As well, the "bricks" material is significantly darker than the textures used in the unique brick floor types. I don't know to what extent this is fixable. It seems like incorporating this into Stuffed Floors well would destroy some of the signature distinctiveness of the red-brick and white-grey mortar which fertile fields is going for. The answer (if possible) might be to just make it so that bricks can't be used to make Stuffed Floors.

Alternatively, if the author of Fertile Fields says its okay it might be possible to make a custom stuff-coloring mask, fix the color of the bricks material to be the lighter red, and then replace the existing simple "Bricks" in Stuffed Floors with the new one with mortar in between, but stuffable. Or just supplement the existing "simple bricks" with "mortared bricks" and "mortared herringbone." That could be cool, and allow a range of new styles with various stone colorings and the mortar. It'd still have the issue of "brick flagstone" and such though, unless there's an easy way to mark on a floortype def like "except this can't be made of this specific material."

Screenshot 2020-10-07 195724

This picture shows how different "bricks material with stuffed floor floor types" (top) is from the native "bricks" from fertile floors. Actually

@InsanityPrelude
Copy link
Contributor

On second thought, a bit of further analysis and testing shows that I don't think there's an easy fix for making "Bright red bricks" by changing colors or anything. And the floor texture for the brick floors is 1024x1024, mostly because of the mortar gradient in the background. Just changing the color of the bricks material doesn't do a lot to make it lighter, because the base texture of the floor types in Stuffed Floors is quite a bit darker before coloring to begin with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants