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

Have beds to depend necessarily on red dye? #475

Closed
Echoes91 opened this issue Mar 25, 2015 · 47 comments
Closed

Have beds to depend necessarily on red dye? #475

Echoes91 opened this issue Mar 25, 2015 · 47 comments

Comments

@Echoes91
Copy link

We know that colored beds are nicer, and also that red wool should come out from red dye, but why should somebody stay awake every night until he finds a rose? The bed should have worked the same whatever the color of the wool was.
I can imagine that defining and drawing textures for each color like the good old PilzAdam's work may be boring and low priority, that's why i have two proposals: either fall back to the Minecraftish solution where every wool color gives the same red bed (it may be one of the worst things to copy but after all why the bed is red if not due to MC?), or we can make the bed white. White bed could be crafted coherently with white wool, thus not depending on any flower species, and also look more sober (I think MT feels overall more sober than MC).
What do you think? If you like either one or the other one i can make a PR.
Sorry for the long writing...

@C1ffisme
Copy link

IDEA : Why not make the simple bed white, and the fancy bed red? Then the simple bed can be used for simple sleeping, and the fancy one for decoration when you find a rose!

@Echoes91
Copy link
Author

That can be a good compromise too.

@C1ffisme
Copy link

The problem is making a white bed that looks good. I don't think I would want to sleep in a white bed. So someone has to make a REALLY good texture.

Or, since leaves are pretty much everywhere, why not make leaves (or cactus) make a green dye, which can make the simple bed.

@0-afflatus
Copy link

The Way To Go is different coloured beds like the old-style beds.

@C1ffisme
Copy link

C1ffisme commented Apr 7, 2015

@0-afflatus True, but then we have to add a whole bunch of nodes. And people really don't need that many in-game. It's better to have 1 or 2 beds and stick with that.

Maybe if we had a bed that could be dyed via crafting, while the colored types didn't appear in the creative inventory. Maybe different colors of bed could be defined by a param. We just need a non-heavy, not-to-many-node solution.

Or we could go the minecraft way, where every wool makes a bed. Cheap, but might work just for gameplay reasons.

@PilzAdam
Copy link
Contributor

PilzAdam commented Apr 7, 2015

@C1ffisme Making all wool result in the same bed would break my beds mod, since the generic recipe would override the color-specific ones.

@C1ffisme
Copy link

C1ffisme commented Apr 7, 2015

@PilzAdam I believe your beds mod would override the minetest one, overriding the crafting recipe. So actually nothing would happen to your mod, because of naming conventions.

@0-afflatus
Copy link

Personally, the old coloured bed models with the new functionality would suit me fine. And I might just do that if it is not too complicated. I'm not sure this discussion is really going anywhere.

@PilzAdam
Copy link
Contributor

PilzAdam commented Apr 8, 2015

@C1ffisme this is not specified behavior, so we have to assume the worst case. Maybe add a way in the engine to remove / override recipes.

@C1ffisme
Copy link

C1ffisme commented Apr 8, 2015

@PilzAdam I saw an engine issue about that.

@npmaier
Copy link

npmaier commented Apr 8, 2015

I like white beds. It kind of harken back to days on the farm where they
had white mauslin sheets. I've seen pictures of the depression. Early
survival would be more authentic
-Bikemare
On Apr 8, 2015 12:17 PM, "C1ffisme" notifications@github.com wrote:

@PilzAdam https://github.com/PilzAdam I saw an engine issue about that.

Reply to this email directly or view it on GitHub
#475 (comment)
.

@BlockMen
Copy link
Contributor

BlockMen commented Apr 8, 2015

@PilzAdam huh? Except some 0.4.5 or 0.4.6 dev version user mods with the same name did always replace the game mods...
And IMO its the right behavior.

@PilzAdam
Copy link
Contributor

PilzAdam commented Apr 8, 2015

@BlockMen why do you provide an API to add new beds when mods that want to add new beds have to override it?

@BlockMen
Copy link
Contributor

BlockMen commented Apr 9, 2015

@PilzAdam i ment there general behavior of mods.
And just for the protocol: Why are you claiming the whole time your mod would break when you are not even using the API...

@PilzAdam
Copy link
Contributor

PilzAdam commented Apr 9, 2015

@BlockMen it's not about my mod. A mod that wants to add beds for all colors using the API you provide would depend on unspecified behavior regarding the crafting recipes.

@Echoes91
Copy link
Author

Echoes91 commented Apr 9, 2015

What about giving it a different name? "morebeds" could add new ones depending on default "beds" without overriding it, as moreores does not override default ores.
Otherwise you could merge both good works by adding the old colored beds to the new code and API within the same "beds" mod, right?

@BlockMen
Copy link
Contributor

@PilzAdam you can use any recipe. only the recipe for the default beds is fixed.

@Echoes91 I don't see the point in renaming. The reason that there are "only" two beds is that Minetest Game should not get bloated. If ppl want more colors, a mods could easily add by using the API.

And yeah, one solution would be a white bed or a "raw" bed that could get crafted with any kind of wool and dyed afterwards. But first i think a white bed wouldnt look good and second it would be even more confusing than now IMO.

@Echoes91
Copy link
Author

I meant to give a different name to the mod that should add more beds, so that it can use the default mod's API without overriding it. And the raw/white bed should be crafted only with the raw white wool, without overriding any colored bed or creating any confusion.

@C1ffisme
Copy link

I wonder... what if we could set the texture of the bed based on a parameter, and not make several nodes of beds?

I'm not sure if this is possible, but if this could be done, you can just see all the node types we could save. Wool could be 1 node type, metal blocks could be 1 node type, desert and normal stone could be 1 node type, we would just need two things:

  • Make the bed item have data which corresponds to either a string with a texture, or a number with a colorize hue.
  • Set the bed depending on what data this item has, and set the drop so that the item has the data of the bed.

You can see how useful this feature would be in mods.

@BlockMen
Copy link
Contributor

Removed color dependency in Minetest NeXt.
BlockMen/minetest_next@4ff5cad

@paramat
Copy link
Contributor

paramat commented Mar 8, 2016

I realised this is not much of an issue, since to get wool you need cotton seeds from junglegrass, and jungles are usually more distant than roses.
Beds are therefore an acheivement, so i don't mind them being a challenge to create.

@0-afflatus
Copy link

OK I'm going to look for a different solution for night-skip voting as using the beds mod from minetest_game in grailtest isn't satisfying my requirements. AFA decor is concerned, players can do interesting things with coloured wool. I think I want to de-couple night-skip voting from this.

I'll stop making comments on this issue as minetest_game isn't really my concern.

@paramat
Copy link
Contributor

paramat commented May 25, 2017

I'm thinking either a range of coloured beds, plus a white one needing no dye, as beds are part of a builds furniture.
Or, just a white bed needing no dye. At least then the basic bed is a neutral colour.
Coloured beds could perhaps be left to mods.

@paramat
Copy link
Contributor

paramat commented May 25, 2017

to get wool you need cotton seeds from junglegrass, and jungles are usually more distant than roses.

This is why the difficulty argument is no issue, not needing a dye doesn't really make them easier to craft. The locations of biomes means that you are highly likely to pass a rose by the time you find a jungle for cotton seeds.
I don't like how we are so obviously copying MC by having a red bed.

I suggest a white bed from:

output = "beds:bed",
recipe = {
{"group:wool", "group:wool", "group:wool"},
{"group:wood", "group:wood", "group:wood"}
},
})

minetest.register_craft({
output = "beds:fancy_bed",
recipe = {
{"", "", "group:stick"},
{"group:wool", "group:wool", "group:wool"},
{"group:wood", "group:wood", "group:wood"},
},
})

I will probably make the texture and a PR soon.

@mtango688
Copy link

Sweet! Thanks a lot!

@paramat
Copy link
Contributor

paramat commented Jun 3, 2017

Thinking on this, i'm sure many like the red beds, so the thing to do would be to add white beds (or maybe just the simple one) without removing the red ones.

@paramat paramat self-assigned this Jun 10, 2017
@paramat
Copy link
Contributor

paramat commented Jun 15, 2017

I'll register a 'simple white bed' soon, any comments or objections?

@C1ffisme
Copy link

Just feel like mentioning it... but MC recently added colored beds to their game. Although even they didn't have enough block IDs for it, and the colored beds are actually tile entities...

@paramat
Copy link
Contributor

paramat commented Jun 23, 2017

We are considering that, there's a PR that might be reworked to do that but the author is absent, so in the meantime this will be a simple quick fix.

@paramat
Copy link
Contributor

paramat commented Jun 23, 2017

See #1794 i will therefore not work on a white bed. Coloured beds are useful for colour co-ordination in builds.

@paramat
Copy link
Contributor

paramat commented Jul 5, 2018

PR #2150

@paramat
Copy link
Contributor

paramat commented Sep 14, 2018

More opinions needed on a solution, @Ezhh ?

@Ezhh
Copy link
Contributor

Ezhh commented Sep 14, 2018

I'm rather indifferent about beds in general, and wondering why some people are against adding a white bed.

I do feel adding all colours isn't necessary for MTG itself.

@paramat
Copy link
Contributor

paramat commented Feb 8, 2019

How about making the red bed not require red dye as a simple fix? After all most of the bed is the mattress which can be considered white wool. We have many items with coloured components that don't use a dye.
Have to consider mod recipe conflicts though.

Recipe conflicts:
'coloured beds' by Napiophelios, however the 2 bed mods wouldn't be used together.

@Ezhh
Copy link
Contributor

Ezhh commented Feb 9, 2019

No objection to removing the requirement for dye.

I'm not familiar with the mod you're referring to, but most mods are intended for use with MTG, so please don't rule that out unless you've checked (though generally we shouldn't be restricted based on what another non-MTG mod has done, it's still good practice not to break something that already exists when we're aware of a possible issue).

@paramat
Copy link
Contributor

paramat commented Feb 9, 2019

Hmm, in this case the mod is very much MTG beds mod code modified, instead of something meant to be parallel, so i think it's the responsibility of the mod to work around MTG. If we merge my suggestion i'll make a post in the mod thread.

https://forum.minetest.net/viewtopic.php?t=18775
"This is a modified version of the default Minetest game's beds mod by Blockmen.
These are experimental replacements for the beds mod.
Don't test them except for in a newly created world."
Also experimental so i'll go ahead and make a PR.

@paramat
Copy link
Contributor

paramat commented Feb 9, 2019

PR #2309

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