[Testing] Try to fix text glitches.

description.txt

@@ -1 +0,0 @@
-Adds signs with readable text.

init.lua

@@ -9,8 +9,6 @@
 --		{ delta = {entity position for 270° yaw}, exact yaw expression }
 --		{ delta = {entity position for 90° yaw}, exact yaw expression }
 -- }
--- Made colored metal signs optionals
-local enable_colored_metal_signs = true
 
 -- CWz's keyword interact mod uses this setting.
 local current_keyword = minetest.setting_get("interact_keyword") or "iaccept"
@@ -33,8 +31,6 @@ end
 
 signs_lib.modpath = minetest.get_modpath("signs_lib")
 
-local DEFAULT_TEXT_SCALE = {x=0.8, y=0.5}
-
 signs_lib.regular_wall_sign_model = {
 	nodebox = {
 		type = "wallmounted",
@@ -45,10 +41,10 @@ signs_lib.regular_wall_sign_model = {
 	textpos = {
 		nil,
 		nil,
-		{delta = { x =  0.41, y = 0.07, z =  0    }, yaw = math.pi / -2},
-		{delta = { x = -0.41, y = 0.07, z =  0    }, yaw = math.pi / 2},
-		{delta = { x =  0,    y = 0.07, z =  0.41 }, yaw = 0},
-		{delta = { x =  0,    y = 0.07, z = -0.41 }, yaw = math.pi},
+		{delta = {x =  0.43,  y = 0.07, z =  0     }, yaw = math.pi / -2},
+		{delta = {x = -0.43,  y = 0.07, z =  0     }, yaw = math.pi / 2},
+		{delta = {x =  0,     y = 0.07, z =  0.43  }, yaw = 0},
+		{delta = {x =  0,     y = 0.07, z = -0.43  }, yaw = math.pi},
 	}
 }
 
@@ -58,10 +54,10 @@ signs_lib.metal_wall_sign_model = {
 		fixed = {-0.4375, -0.25, 0.4375, 0.4375, 0.375, 0.5}
 	},
 	textpos = {
-		{delta = { x =  0,     y = 0.07, z =  0.41 }, yaw = 0},
-		{delta = { x =  0.41,  y = 0.07, z =  0    }, yaw = math.pi / -2},
-		{delta = { x =  0,     y = 0.07, z = -0.41 }, yaw = math.pi},
-		{delta = { x = -0.41,  y = 0.07, z =  0    }, yaw = math.pi / 2},
+		{delta = {x =  0,     y = 0.07, z =  0.43  }, yaw = 0},
+		{delta = {x =  0.43,  y = 0.07, z =  0     }, yaw = math.pi / -2},
+		{delta = {x =  0,     y = 0.07, z = -0.43  }, yaw = math.pi},
+		{delta = {x = -0.43,  y = 0.07, z =  0     }, yaw = math.pi / 2},
 	}
 }
 
@@ -74,10 +70,10 @@ signs_lib.yard_sign_model = {
 		}
 	},
 	textpos = {
-		{delta = { x =  0,    y = 0.07, z = -0.08 }, yaw = 0},
-		{delta = { x = -0.08, y = 0.07, z =  0    }, yaw = math.pi / -2},
-		{delta = { x =  0,    y = 0.07, z =  0.08 }, yaw = math.pi},
-		{delta = { x =  0.08, y = 0.07, z =  0    }, yaw = math.pi / 2},
+		{delta = {x =  0,      y = 0.07, z = -0.068}, yaw = 0},
+		{delta = {x = -0.068,  y = 0.07, z =  0    }, yaw = math.pi / -2},
+		{delta = {x =  0,      y = 0.07, z =  0.068}, yaw = math.pi},
+		{delta = {x =  0.068,  y = 0.07, z =  0    }, yaw = math.pi / 2},
 	}
 }
 
@@ -90,10 +86,10 @@ signs_lib.hanging_sign_model = {
 		}
 	},
 	textpos = {
-		{delta = { x =  0,    y = -0.02, z = -0.08 }, yaw = 0},
-		{delta = { x = -0.08, y = -0.02, z =  0    }, yaw = math.pi / -2},
-		{delta = { x =  0,    y = -0.02, z =  0.08 }, yaw = math.pi},
-		{delta = { x =  0.08, y = -0.02, z =  0    }, yaw = math.pi / 2},
+		{delta = {x =  0,      y = -0.02, z = -0.063}, yaw = 0},
+		{delta = {x = -0.063,  y = -0.02, z =  0    }, yaw = math.pi / -2},
+		{delta = {x =  0,      y = -0.02, z =  0.063}, yaw = math.pi},
+		{delta = {x =  0.063,  y = -0.02, z =  0    }, yaw = math.pi / 2},
 	}
 }
 
@@ -106,10 +102,10 @@ signs_lib.sign_post_model = {
 		}
 	},
 	textpos = {
-		{delta = { x = 0,    y = 0.07, z = -0.2 }, yaw = 0},
-		{delta = { x = -0.2, y = 0.07, z = 0    }, yaw = math.pi / -2},
-		{delta = { x = 0,    y = 0.07, z = 0.2  }, yaw = math.pi},
-		{delta = { x = 0.2,  y = 0.07, z = 0    }, yaw = math.pi / 2},
+		{delta = {x = 0,      y = 0.07, z = -0.188}, yaw = 0},
+		{delta = {x = -0.188, y = 0.07, z = 0     }, yaw = math.pi / -2},
+		{delta = {x = 0,      y = 0.07, z = 0.188 }, yaw = math.pi},
+		{delta = {x = 0.188,  y = 0.07, z = 0     }, yaw = math.pi / 2},
 	}
 }
 
@@ -120,19 +116,18 @@ signs_lib.gettext = S
 -- the list of standard sign nodes
 
 signs_lib.sign_node_list = {
-	"default:sign_wall_wood",
-	"default:sign_wall_steel",
-	"signs:sign_yard",
-	"signs:sign_hanging",
-	"signs:sign_wall_green",
-	"signs:sign_wall_yellow",
-	"signs:sign_wall_red",
-	"signs:sign_wall_white_red",
-	"signs:sign_wall_white_black",
-	"signs:sign_wall_orange",
-	"signs:sign_wall_blue",
-	"signs:sign_wall_brown",
-	"locked_sign:sign_wall_locked"
+		"default:sign_wall_wood",
+		"signs:sign_yard",
+		"signs:sign_hanging",
+		"signs:sign_wall_green",
+		"signs:sign_wall_yellow",
+		"signs:sign_wall_red",
+		"signs:sign_wall_white_red",
+		"signs:sign_wall_white_black",
+		"signs:sign_wall_orange",
+		"signs:sign_wall_blue",
+		"signs:sign_wall_brown",
+		"locked_sign:sign_wall_locked"
 }
 
 local default_sign, default_sign_image
@@ -146,9 +141,6 @@ else
 	default_sign_image = "default_sign_wall.png"
 end
 
-default_sign_metal = "default:sign_wall_steel"
-default_sign_metal_image = "default_sign_steel.png"
-
 --table copy
 
 function signs_lib.table_copy(t)
@@ -165,7 +157,7 @@ end
 
 -- infinite stacks
 
-if not minetest.setting_getbool("creative_mode") then
+if minetest.get_modpath("unified_inventory") or not minetest.setting_getbool("creative_mode") then
 	signs_lib.expect_infinite_stacks = false
 else
 	signs_lib.expect_infinite_stacks = true
@@ -184,6 +176,8 @@ local FONT_FMT_SIMPLE = "hdf_%02x.png"
 -- Path to the textures.
 local TP = MP.."/textures"
 
+local TEXT_SCALE = {x=0.8, y=0.5}
+
 -- Lots of overkill here. KISS advocates, go away, shoo! ;) -- kaeza
 
 local PNG_HDR = string.char(0x89, 0x50, 0x4E, 0x47, 0x0D, 0x0A, 0x1A, 0x0A)
@@ -307,16 +301,14 @@ local function fill_line(x, y, w, c)
 	return table.concat(tex)
 end
 
-local function make_line_texture(line, lineno, pos)
+local function make_line_texture(line, lineno)
 
 	local width = 0
 	local maxw = 0
 
 	local words = { }
-	local n = minetest.registered_nodes[minetest.get_node(pos).name]
-	local default_color = n.default_color or 0
 
-	local cur_color = tonumber(default_color, 16)
+	local cur_color = 0
 
 	-- We check which chars are available here.
 	for word_i, word in ipairs(line) do
@@ -397,12 +389,12 @@ local function make_line_texture(line, lineno, pos)
 	return table.concat(texture), lineno
 end
 
-local function make_sign_texture(lines, pos)
+local function make_sign_texture(lines)
 	local texture = { ("[combine:%dx%d"):format(SIGN_WIDTH, LINE_HEIGHT * NUMBER_OF_LINES) }
 	local lineno = 0
 	for i = 1, #lines do
 		if lineno >= NUMBER_OF_LINES then break end
-		local linetex, ln = make_line_texture(lines[i], lineno, pos)
+		local linetex, ln = make_line_texture(lines[i], lineno)
 		table.insert(texture, linetex)
 		lineno = ln + 1
 	end
@@ -410,13 +402,11 @@ local function make_sign_texture(lines, pos)
 	return table.concat(texture, "")
 end
 
-local function set_obj_text(obj, text, new, pos)
+local function set_obj_text(obj, text, new)
 	local split = new and split_lines_and_words or split_lines_and_words_old
-	local n = minetest.registered_nodes[minetest.get_node(pos).name]
-	local text_scale = n.text_scale or DEFAULT_TEXT_SCALE
 	obj:set_properties({
-		textures={make_sign_texture(split(text), pos)},
-		visual_size = text_scale,
+		textures={make_sign_texture(split(text))},
+		visual_size = TEXT_SCALE,
 	})
 end
 
@@ -497,28 +487,15 @@ signs_lib.update_sign = function(pos, fields, owner)
 	end
 	local text = meta:get_string("text")
 	if text == nil then return end
-	local objects = minetest.get_objects_inside_radius(pos, 0.5)
-	for _, v in ipairs(objects) do
-		local e = v:get_luaentity()
-		if e and e.name == "signs:text" then
-			v:remove()
-		end
-	end
 
-	-- if there is no entity
 	local sign_info
 	local signnode = minetest.get_node(pos)
-	local signname = signnode.name
-	local textpos = minetest.registered_nodes[signname].textpos
-	if textpos then
-		sign_info = textpos[minetest.get_node(pos).param2 + 1]
-	elseif signnode.name == "signs:sign_yard" then
+	if signnode.name == "signs:sign_yard" then
 		sign_info = signs_lib.yard_sign_model.textpos[minetest.get_node(pos).param2 + 1]
 	elseif signnode.name == "signs:sign_hanging" then
 		sign_info = signs_lib.hanging_sign_model.textpos[minetest.get_node(pos).param2 + 1]
 	elseif string.find(signnode.name, "sign_wall") then
 		if signnode.name == default_sign
-		  or signnode.name == default_sign_metal
 		  or signnode.name == "locked_sign:sign_wall_locked" then
 			sign_info = signs_lib.regular_wall_sign_model.textpos[minetest.get_node(pos).param2 + 1]
 		else
@@ -530,10 +507,33 @@ signs_lib.update_sign = function(pos, fields, owner)
 	if sign_info == nil then
 		return
 	end
-	local text = minetest.add_entity({x = pos.x + sign_info.delta.x,
-										y = pos.y + sign_info.delta.y,
-										z = pos.z + sign_info.delta.z}, "signs:text")
-	text:setyaw(sign_info.yaw)
+
+	local objects = minetest.get_objects_inside_radius(pos, 0.5)
+	local sign_obj
+	for _, v in ipairs(objects) do
+		local e = v:get_luaentity()
+		if e and e.name == "signs:text" then
+			if sign_obj then
+				v:remove()
+			else
+				set_obj_text(v, text, new)
+				sign_obj = v
+			end
+		end
+	end
+
+	-- if there is no entity, create one
+	if not sign_obj then
+		sign_obj = minetest.add_entity(pos, "signs:text")
+	end
+
+	-- set pos and yaw here. solves issue #2
+	sign_obj:setpos({
+		x = pos.x + sign_info.delta.x,
+		y = pos.y + sign_info.delta.y,
+		z = pos.z + sign_info.delta.z,
+	})
+	sign_obj:setyaw(sign_info.yaw)
 end
 
 -- What kind of sign do we need to place, anyway?
@@ -563,7 +563,7 @@ function signs_lib.determine_sign_type(itemstack, placer, pointed_thing, locked)
 	local node=minetest.get_node(pointed_thing.under)
 
 	if minetest.registered_nodes[node.name] and minetest.registered_nodes[node.name].on_rightclick then
-		return minetest.registered_nodes[node.name].on_rightclick(pointed_thing.under, node, placer, itemstack, pointed_thing)
+		return minetest.registered_nodes[node.name].on_rightclick(pointed_thing.under, node, placer, itemstack)
 	else
 		local above = pointed_thing.above
 		local under = pointed_thing.under
@@ -583,6 +583,7 @@ function signs_lib.determine_sign_type(itemstack, placer, pointed_thing, locked)
 		end
 
 		local fdir = minetest.dir_to_facedir(dir)
+
 		local pt_name = minetest.get_node(under).name
 		local signname = itemstack:get_name()
 
@@ -592,11 +593,8 @@ function signs_lib.determine_sign_type(itemstack, placer, pointed_thing, locked)
 			minetest.add_node(above, {name = "signs:sign_hanging", param2 = fdir})
 		elseif wdir == 1 and signname == default_sign then
 			minetest.add_node(above, {name = "signs:sign_yard", param2 = fdir})
-		elseif signname == default_sign_metal then
-			minetest.add_node(above, {name = signname, param2 = wdir })
 		elseif signname ~= default_sign
-		  and signname ~= default_sign_metal
-		  and signname ~= "locked_sign:sign_wall_locked" then -- it's a signs_lib colored metal wall sign.
+		  and signname ~= "locked_sign:sign_wall_locked" then -- it's a metal wall sign.
 			minetest.add_node(above, {name = signname, param2 = fdir})
 		else -- it must be a default or locked wooden wall sign
 			minetest.add_node(above, {name = signname, param2 = wdir }) -- note it's wallmounted here!
@@ -801,22 +799,30 @@ minetest.register_node(":locked_sign:sign_wall_locked", {
 	on_rotate = signs_lib.wallmounted_rotate
 })
 
--- default metal sign, if defined
+-- metal, colored signs
 
-if minetest.registered_nodes["default:sign_wall_steel"] then
-	minetest.register_node(":"..default_sign_metal, {
-		description = S("Sign"),
-		inventory_image = default_sign_metal_image,
-		wield_image = default_sign_metal_image,
+local sign_colors = { "green", "yellow", "red", "white_red", "white_black", "orange", "blue", "brown" }
+
+for _, color in ipairs(sign_colors) do
+	minetest.register_node(":signs:sign_wall_"..color, {
+		description = S("Sign ("..color..", metal)"),
+		inventory_image = "signs_"..color.."_inv.png",
+		wield_image = "signs_"..color.."_inv.png",
 		node_placement_prediction = "",
-		sunlight_propagates = true,
 		paramtype = "light",
-		paramtype2 = "wallmounted",
+		sunlight_propagates = true,
+		paramtype2 = "facedir",
 		drawtype = "nodebox",
-		node_box = signs_lib.regular_wall_sign_model.nodebox,
-		tiles = {"signs_wall_sign_metal.png"},
+		node_box = signs_lib.metal_wall_sign_model.nodebox,
+		tiles = {
+			"signs_metal_tb.png",
+			"signs_metal_tb.png",
+			"signs_metal_sides.png",
+			"signs_metal_sides.png",
+			"signs_metal_back.png",
+			"signs_"..color.."_front.png"
+		},
 		groups = sign_groups,
-
 		on_place = function(itemstack, placer, pointed_thing)
 			return signs_lib.determine_sign_type(itemstack, placer, pointed_thing)
 		end,
@@ -832,65 +838,18 @@ if minetest.registered_nodes["default:sign_wall_steel"] then
 		on_punch = function(pos, node, puncher)
 			signs_lib.update_sign(pos)
 		end,
-		on_rotate = signs_lib.wallmounted_rotate
 	})
 end
 
--- metal, colored signs
-if enable_colored_metal_signs then
-	local sign_colors = { "green", "yellow", "red", "white_red", "white_black", "orange", "blue", "brown" }
-	local sign_default_text_colors = { "f", "0", "f", "4", "0", "0", "f", "f" }
-
-	for i, color in ipairs(sign_colors) do
-		minetest.register_node(":signs:sign_wall_"..color, {
-			description = S("Sign ("..color..", metal)"),
-			inventory_image = "signs_"..color.."_inv.png",
-			wield_image = "signs_"..color.."_inv.png",
-			node_placement_prediction = "",
-			paramtype = "light",
-			sunlight_propagates = true,
-			paramtype2 = "facedir",
-			drawtype = "nodebox",
-			node_box = signs_lib.metal_wall_sign_model.nodebox,
-			tiles = {
-				"signs_metal_tb.png",
-				"signs_metal_tb.png",
-				"signs_metal_sides.png",
-				"signs_metal_sides.png",
-				"signs_metal_back.png",
-				"signs_"..color.."_front.png"
-			},
-			default_color = sign_default_text_colors[i],
-			groups = sign_groups,
-			on_place = function(itemstack, placer, pointed_thing)
-				return signs_lib.determine_sign_type(itemstack, placer, pointed_thing)
-			end,
-			on_construct = function(pos)
-				signs_lib.construct_sign(pos)
-			end,
-			on_destruct = function(pos)
-				signs_lib.destruct_sign(pos)
-			end,
-			on_receive_fields = function(pos, formname, fields, sender)
-				signs_lib.receive_fields(pos, formname, fields, sender)
-			end,
-			on_punch = function(pos, node, puncher)
-				signs_lib.update_sign(pos)
-			end,
-		})
-	end
-end
-
 local signs_text_on_activate
 
 signs_text_on_activate = function(self)
-	local pos = self.object:getpos()
-	local meta = minetest.get_meta(pos)
+	local meta = minetest.get_meta(self.object:getpos())
 	local text = meta:get_string("text")
 	local new = (meta:get_int("__signslib_new_format") ~= 0)
 	if text then
 		text = trim_input(text)
-		set_obj_text(self.object, text, new, pos)
+		set_obj_text(self.object, text, new)
 	end
 end
 
@@ -934,7 +893,7 @@ function signs_lib.register_fence_with_sign(fencename, fencewithsignname)
 		end
 
 		if def_under and def_under.on_rightclick then
-			return def_under.on_rightclick(pointed_thing.under, node_under, placer, itemstack, pointed_thing) or itemstack
+			return def_under.on_rightclick(pointed_thing.under, node_under, placer, itemstack) or itemstack
 		elseif def_under and def_under.buildable_to then
 			minetest.add_node(pointed_thing.under, {name = fencename, param2 = fdir})
 			if not signs_lib.expect_infinite_stacks then
@@ -971,7 +930,7 @@ function signs_lib.register_fence_with_sign(fencename, fencewithsignname)
 	minetest.register_node(":"..fencename, def)
 	minetest.register_node(":"..fencewithsignname, def_sign)
 	table.insert(signs_lib.sign_node_list, fencewithsignname)
-	minetest.log("verbose", S("Registered %s and %s"):format(fencename, fencewithsignname))
+	minetest.log("debug", S("Registered %s and %s"):format(fencename, fencewithsignname))
 end
 
 build_char_db()
@@ -981,15 +940,13 @@ minetest.register_alias("sign_wall_locked", "locked_sign:sign_wall_locked")
 
 signs_lib.register_fence_with_sign("default:fence_wood", "signs:sign_post")
 
--- restore signs' text after /clearobjects and the like, the next time
--- a block is reloaded by the server.
+-- restore signs' text after /clearobjects and the like
 
-minetest.register_lbm({
+minetest.register_abm({
 	nodenames = signs_lib.sign_node_list,
-	name = "signs_lib:restore_sign_text",
-	label = "Restore sign text",
-	run_at_every_load = true,
-	action = function(pos, node)
+	interval = 15,
+	chance = 1,
+	action = function(pos, node, active_object_count, active_object_count_wider)
 		signs_lib.update_sign(pos)
 	end
 })
@@ -1005,7 +962,7 @@ minetest.register_craft({
 	}
 })
 
---Alternate recipe
+--Alternate recipe.
 
 minetest.register_craft({
     	output = "locked_sign:sign_wall_locked",
@@ -1016,136 +973,134 @@ minetest.register_craft({
 })
 
 -- craft recipes for the metal signs
-if enable_colored_metal_signs then
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_green",
-		recipe = {
-				{ "dye:dark_green", "dye:white", "dye:dark_green" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_green 4",
+        recipe = {
+			{ "dye:dark_green", "dye:white", "dye:dark_green" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_green 2",
-		recipe = {
-				{ "dye:dark_green", "dye:white", "dye:dark_green" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_green 2",
+        recipe = {
+			{ "dye:dark_green", "dye:white", "dye:dark_green" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_yellow",
-		recipe = {
-				{ "dye:yellow", "dye:black", "dye:yellow" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_yellow 4",
+        recipe = {
+			{ "dye:yellow", "dye:black", "dye:yellow" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_yellow 2",
-		recipe = {
-				{ "dye:yellow", "dye:black", "dye:yellow" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_yellow 2",
+        recipe = {
+			{ "dye:yellow", "dye:black", "dye:yellow" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_red",
-		recipe = {
-				{ "dye:red", "dye:white", "dye:red" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_red 4",
+        recipe = {
+			{ "dye:red", "dye:white", "dye:red" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_red 2",
-		recipe = {
-				{ "dye:red", "dye:white", "dye:red" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_red 2",
+        recipe = {
+			{ "dye:red", "dye:white", "dye:red" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_white_red",
-		recipe = {
-				{ "dye:white", "dye:red", "dye:white" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_white_red 4",
+        recipe = {
+			{ "dye:white", "dye:red", "dye:white" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_white_red 2",
-		recipe = {
-				{ "dye:white", "dye:red", "dye:white" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_white_red 2",
+        recipe = {
+			{ "dye:white", "dye:red", "dye:white" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_white_black",
-		recipe = {
-				{ "dye:white", "dye:black", "dye:white" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_white_black 4",
+        recipe = {
+			{ "dye:white", "dye:black", "dye:white" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_white_black 2",
-		recipe = {
-				{ "dye:white", "dye:black", "dye:white" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_white_black 2",
+        recipe = {
+			{ "dye:white", "dye:black", "dye:white" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_orange",
-		recipe = {
-				{ "dye:orange", "dye:black", "dye:orange" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_orange 4",
+        recipe = {
+			{ "dye:orange", "dye:black", "dye:orange" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_orange 2",
-		recipe = {
-				{ "dye:orange", "dye:black", "dye:orange" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_orange 2",
+        recipe = {
+			{ "dye:orange", "dye:black", "dye:orange" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_blue",
-		recipe = {
-				{ "dye:blue", "dye:white", "dye:blue" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_blue 4",
+        recipe = {
+			{ "dye:blue", "dye:white", "dye:blue" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_blue 2",
-		recipe = {
-				{ "dye:blue", "dye:white", "dye:blue" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_blue 2",
+        recipe = {
+			{ "dye:blue", "dye:white", "dye:blue" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_brown",
-		recipe = {
-				{ "dye:brown", "dye:white", "dye:brown" },
-				{ "", default_sign_metal, "" }
-		},
-	})
+minetest.register_craft( {
+        output = "signs:sign_wall_brown 4",
+        recipe = {
+			{ "dye:brown", "dye:white", "dye:brown" },
+			{ "default:steel_ingot", "default:steel_ingot", "default:steel_ingot" }
+        },
+})
 
-	minetest.register_craft( {
-		output = "signs:sign_wall_brown 2",
-		recipe = {
-				{ "dye:brown", "dye:white", "dye:brown" },
-				{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
-		},
-	})
-end
+minetest.register_craft( {
+        output = "signs:sign_wall_brown 2",
+        recipe = {
+			{ "dye:brown", "dye:white", "dye:brown" },
+			{ "steel:sheet_metal", "steel:sheet_metal", "steel:sheet_metal" }
+        },
+})
 
 if minetest.setting_get("log_mods") then
 	minetest.log("action", S("signs loaded"))

mod.conf

@@ -1 +0,0 @@
-name = signs_lib