New design

init.lua

@@ -7,6 +7,7 @@ local searches      = {}
 local recipes_cache = {}
 local usages_cache  = {}
 local fuel_cache    = {}
+
 local toolrepair
 
 local progressive_mode = core.settings:get_bool "craftguide_progressive_mode"
@@ -15,7 +16,6 @@ local autocache = core.settings:get_bool "craftguide_autocache"
 
 local http = core.request_http_api()
 local storage = core.get_mod_storage()
-local singleplayer = core.is_singleplayer()
 
 local reg_items = core.registered_items
 local reg_tools = core.registered_tools
@@ -65,7 +65,6 @@ local FORMSPEC_MINIMAL_VERSION = 3
 
 local ROWS = 9
 local LINES = sfinv_only and 5 or 9
-local IPP = ROWS * LINES
 local WH_LIMIT = 8
 
 local XOFFSET = sfinv_only and 3.83 or 11.2
@@ -111,20 +110,6 @@ local function get_fs_version(name)
 	return info and info.formspec_version or 1
 end
 
-local function outdated(name)
-	local fs = fmt([[
-		size[6.6,1.3]
-		image[0,0;1,1;%s]
-		label[1,0;%s]
-		button_exit[2.8,0.8;1,1;;OK]
-	]],
-	PNG.book,
-	"Your Minetest client is outdated.\n" ..
-	"Get the latest version on minetest.net to use the Crafting Guide.")
-
-	return show_formspec(name, "craftguide", fs)
-end
-
 local function mul_elem(elem, n)
 	local fstr, elems = "", {}
 
@@ -711,10 +696,6 @@ local function is_fav(data)
 	return fav, i
 end
 
-local function check_newline(def)
-	return def and def.description and find(def.description, "\n")
-end
-
 local function get_desc(name)
 	if sub(name, 1, 1) == "_" then
 		name = sub(name, 2)
@@ -832,15 +813,11 @@ local function get_output_fs(data, fs, L)
 		fs[#fs + 1] = fmt("item_image_button[%f,%f;%f,%f;%s;%s;%s]",
 			output_X, Y, 1.1, 1.1, item, _name, "")
 
-
-		local def = reg_items[name]
-
 		local infos = {
-			unknown  = not def or nil,
+			unknown  = not reg_items[name] or nil,
 			burntime = fuel_cache[name],
 			repair   = repairable(name),
 			rarity   = L.rarity,
-			newline  = check_newline(def),
 		}
 
 		if next(infos) then
@@ -947,15 +924,12 @@ local function get_grid_fs(data, fs, rcp, spacing)
 		fs[#fs + 1] = fmt(FMT.item_image_button,
 			X, Y, btn_size, btn_size, item, item, label)
 
-		local def = reg_items[name]
-
 		local infos = {
-			unknown  = not def or nil,
+			unknown  = not reg_items[name] or nil,
 			groups   = groups,
 			burntime = fuel_cache[name],
 			cooktime = cooktime,
 			replace  = replace,
-			newline  = check_newline(def),
 		}
 
 		if next(infos) then
@@ -1110,38 +1084,26 @@ local function make_fs(data)
 
 	fs[#fs + 1] = fmt([[
 		style[filter;border=false]
-		field[0.4,0.2;2.5,1;filter;;%s]
+		field[0.4,0.2;6.45,1;filter;;%s]
 		field_close_on_enter[filter;false]
-		box[0,0;2.4,0.6;#bababa25]
+		box[0,0;6.35,0.6;#bababa25]
 	]],
 	ESC(data.filter))
 
 	fs[#fs + 1] = fmt([[
+		style_type[image;noclip=false]
 		style_type[image_button;border=false]
 		style_type[item_image_button;border=false;bgimg_hovered=%s;bgimg_pressed=%s]
 		style[search;fgimg=%s;fgimg_hovered=%s]
 		style[clear;fgimg=%s;fgimg_hovered=%s]
-		style[prev_page;fgimg=%s;fgimg_hovered=%s;fgimg_pressed=%s]
-		style[next_page;fgimg=%s;fgimg_hovered=%s;fgimg_pressed=%s]
-		style[pagenum;border=false]
 	]],
 	PNG.selected, PNG.selected,
 	PNG.search, PNG.search_hover,
-	PNG.clear, PNG.clear_hover,
+	PNG.clear, PNG.clear_hover)
-	PNG.prev, PNG.prev_hover, PNG.prev_hover,
-	PNG.next, PNG.next_hover, PNG.next_hover)
 
-	fs[#fs + 1] = fmt(mul_elem(FMT.image_button, 4),
+	fs[#fs + 1] = fmt(mul_elem(FMT.image_button, 2),
-		sfinv_only and 2.6 or 2.54, -0.06, 0.85, 0.85, "", "search", "",
+		sfinv_only and 2.6 or 6.45, -0.06, 0.85, 0.85, "", "search", "",
-		sfinv_only and 3.3 or 3.25, -0.06, 0.85, 0.85, "", "clear", "",
+		sfinv_only and 3.3 or 7.15, -0.06, 0.85, 0.85, "", "clear", "")
-		sfinv_only and 5.45 or (9 * 6.83) / 11, -0.06, 0.85, 0.85, "", "prev_page", "",
-		sfinv_only and 7.2  or (9 * 8.75) / 11, -0.06, 0.85, 0.85, "", "next_page", "")
-
-	data.pagemax = max(1, ceil(#data.items / IPP))
-
-	fs[#fs + 1] = fmt(FMT.button,
-		sfinv_only and 5.85 or 6, -0.1, sfinv_only and 1.82 or 1.62, 1, "pagenum",
-		fmt("%u / %u", data.pagenum, data.pagemax))
 
 	if #data.items == 0 then
 		local no_item = ES"No item to show"
@@ -1155,24 +1117,29 @@ local function make_fs(data)
 		fs[#fs + 1] = fmt(FMT.label, pos, 2, no_item)
 	end
 
-	local first_item = (data.pagenum - 1) * IPP
+	fs[#fs + 1] = "scroll_container[0,1.3;9,9.3;scrbar;vertical]"
 
-	for i = first_item, first_item + IPP - 1 do
+	for i = 0, #data.items do
 		local item = data.items[i + 1]
 		if not item then break end
 
 		local X = i % ROWS
-		local Y = (i % IPP - X) / ROWS + 1
+		local Y = (i - X) / ROWS
-		X = X - (X * (sfinv_only and 0.12 or 0.14)) - 0.05
+		X = X - (X * 0.12)
-		Y = Y - (Y * 0.1) - 0.1
+		Y = Y - (Y * 0.02)
 
 		if data.query_item == item then
 			fs[#fs + 1] = fmt(FMT.image, X, Y, 1, 1, PNG.selected)
 		end
 
-		fs[#fs + 1] = fmt("item_image_button[%f,%f;1,1;%s;%s_inv;]", X, Y, item, item)
+		fs[#fs + 1] = fmt("item_image_button[%f,%f;%f,%f;%s;%s_inv;]",
+			X, Y, 1, 1, item, item)
 	end
 
+	fs[#fs + 1] = "scroll_container_end[]"
+	fs[#fs + 1] = fmt("scrollbaroptions[arrows=hide;max=%u]", (#data.items / 8) * 10 - 80)
+	fs[#fs + 1] = fmt("scrollbar[7.2,0.8;0.45,8.1;vertical;scrbar;%u]", data.scrbar)
+
 	if (data.recipes and #data.recipes > 0) or (data.usages and #data.usages > 0) then
 		get_panels(data, fs)
 	end
@@ -1191,12 +1158,12 @@ end
 
 craftguide.register_craft_type("digging", {
 	description = ES"Digging",
-	icon = "craftguide_steelpick.png",
+	icon = "default_tool_steelpick.png",
 })
 
 craftguide.register_craft_type("digging_chance", {
 	description = ES"Digging Chance",
-	icon = "craftguide_mesepick.png",
+	icon = "default_tool_mesepick.png",
 })
 
 local function search(data)
@@ -1549,7 +1516,7 @@ end
 local function init_data(name)
 	pdata[name] = {
 		filter     = "",
-		pagenum    = 1,
+		scrbar     = 0,
 		items      = init_items,
 		items_raw  = init_items,
 		favs       = {},
@@ -1559,7 +1526,7 @@ end
 
 local function reset_data(data)
 	data.filter      = ""
-	data.pagenum     = 1
+	data.scrbar      = 0
 	data.rnum        = 1
 	data.unum        = 1
 	data.query_item  = nil
@@ -1574,10 +1541,6 @@ on_mods_loaded(get_init_items)
 on_joinplayer(function(player)
 	local name = player:get_player_name()
 	init_data(name)
-
-	if pdata[name].fs_version < FORMSPEC_MINIMAL_VERSION then
-		outdated(name)
-	end
 end)
 
 local function fields(player, _f)
@@ -1605,19 +1568,9 @@ local function fields(player, _f)
 		if data.filter == str then return end
 
 		data.filter = str
-		data.pagenum = 1
+		data.scrbar = 0
 		search(data)
 
-	elseif _f.prev_page or _f.next_page then
-		if data.pagemax == 1 then return end
-		data.pagenum = data.pagenum - (_f.prev_page and 1 or -1)
-
-		if data.pagenum > data.pagemax then
-			data.pagenum = 1
-		elseif data.pagenum == 0 then
-			data.pagenum = data.pagemax
-		end
-
 	elseif _f.fav then
 		local fav, i = is_fav(data)
 		local total = #data.favs
@@ -1665,6 +1618,7 @@ local function fields(player, _f)
 		data.usages     = usages
 		data.rnum       = 1
 		data.unum       = 1
+		data.scrbar     = tonumber(match(_f.scrbar, "%d+"))
 	end
 
 	return true, show_fs(player, name)
@@ -1682,8 +1636,9 @@ if sfinv_only then
 		get = function(self, player, context)
 			local name = player:get_player_name()
 			local data = pdata[name]
+			local formspec = make_fs(data)
 
-			return sfinv.make_formspec(player, context, make_fs(data))
+			return sfinv.make_formspec(player, context, formspec)
 		end,
 
 		on_enter = function(self, player, context)
@@ -1712,7 +1667,17 @@ else
 		local data = pdata[name]
 
 		if data.fs_version < FORMSPEC_MINIMAL_VERSION then
-			return outdated(name)
+			local fs = fmt([[
+				size[6.6,1.3]
+				image[0,0;1,1;%s]
+				label[1,0;%s]
+				button_exit[2.8,0.8;1,1;;OK]
+			]],
+			PNG.nothing,
+			"Your Minetest client is outdated.\n" ..
+			"Get the latest version on minetest.net to use the Crafting Guide.")
+
+			return show_formspec(name, "craftguide", fs)
 		end
 
 		if next(recipe_filters) then
@@ -1743,12 +1708,7 @@ else
 		paramtype = "light",
 		paramtype2 = "wallmounted",
 		sunlight_propagates = true,
-		groups = {
+		groups = {choppy = 1, attached_node = 1, oddly_breakable_by_hand = 1, flammable = 3},
-			choppy = 1,
-			attached_node = 1,
-			oddly_breakable_by_hand = 1,
-			flammable = 3,
-		},
 		node_box = {
 			type = "wallmounted",
 			wall_top    = {-0.5, 0.4375, -0.5, 0.5, 0.5, 0.5},
@@ -1793,7 +1753,7 @@ else
 	if rawget(_G, "sfinv_buttons") then
 		sfinv_buttons.register_button("craftguide", {
 			title = S"Crafting Guide",
-			tooltip = S"Shows a list of available crafting recipes",
+			tooltip = S"Shows a list of available crafting recipes, cooking recipes and fuels",
 			image = PNG.book,
 			action = function(player)
 				on_use(player)
@@ -1803,6 +1763,7 @@ else
 end
 
 if progressive_mode then
+	local PLAYERS = {}
 	local POLL_FREQ = 0.25
 	local HUD_TIMER_MAX = 1.5
 
@@ -1889,34 +1850,6 @@ if progressive_mode then
 		return inv_items
 	end
 
-	local function init_hud(player, data)
-		data.hud = {
-			bg = player:hud_add{
-				hud_elem_type = "image",
-				position      = {x = 0.78, y = 1},
-				alignment     = {x = 1,    y = 1},
-				scale         = {x = 370,  y = 112},
-				text          = PNG.bg,
-			},
-
-			book = player:hud_add{
-				hud_elem_type = "image",
-				position      = {x = 0.79, y = 1.02},
-				alignment     = {x = 1,    y = 1},
-				scale         = {x = 4,    y = 4},
-				text          = PNG.book,
-			},
-
-			text = player:hud_add{
-				hud_elem_type = "text",
-				position      = {x = 0.84, y = 1.04},
-				alignment     = {x = 1,    y = 1},
-				number        = 0xffffff,
-				text          = "",
-			},
-		}
-	end
-
 	local function show_hud_success(player, data)
 		-- It'd better to have an engine function `hud_move` to only need
 		-- 2 calls for the notification's back and forth.
@@ -1964,9 +1897,8 @@ if progressive_mode then
 	-- Workaround. Need an engine call to detect when the contents of
 	-- the player inventory changed, instead.
 	local function poll_new_items()
-		local players = get_players()
+		for i = 1, #PLAYERS do
-		for i = 1, #players do
+			local player = PLAYERS[i]
-			local player = players[i]
 			local name   = player:get_player_name()
 			local data   = pdata[name]
 
@@ -1999,13 +1931,12 @@ if progressive_mode then
 	poll_new_items()
 
 	globalstep(function()
-		local players = get_players()
+		for i = 1, #PLAYERS do
-		for i = 1, #players do
+			local player = PLAYERS[i]
-			local player = players[i]
 			local name   = player:get_player_name()
 			local data   = pdata[name]
 
-			if data.show_hud ~= nil and singleplayer then
+			if data.show_hud ~= nil then
 				show_hud_success(player, data)
 			end
 		end
@@ -2014,6 +1945,8 @@ if progressive_mode then
 	craftguide.add_recipe_filter("Default progressive filter", progressive_filter)
 
 	on_joinplayer(function(player)
+		PLAYERS = get_players()
+
 		local name = player:get_player_name()
 		local data = pdata[name]
 
@@ -2021,9 +1954,31 @@ if progressive_mode then
 		data.inv_items = dslz(meta:get_string "inv_items") or {}
 		data.known_recipes = dslz(meta:get_string "known_recipes") or 0
 
-		if singleplayer then
+		data.hud = {
-			init_hud(player, data)
+			bg = player:hud_add{
-		end
+				hud_elem_type = "image",
+				position      = {x = 0.78, y = 1},
+				alignment     = {x = 1,    y = 1},
+				scale         = {x = 370,  y = 112},
+				text          = PNG.bg,
+			},
+
+			book = player:hud_add{
+				hud_elem_type = "image",
+				position      = {x = 0.79, y = 1.02},
+				alignment     = {x = 1,    y = 1},
+				scale         = {x = 4,    y = 4},
+				text          = PNG.book,
+			},
+
+			text = player:hud_add{
+				hud_elem_type = "text",
+				position      = {x = 0.84, y = 1.04},
+				alignment     = {x = 1,    y = 1},
+				number        = 0xffffff,
+				text          = "",
+			},
+		}
 	end)
 
 	local to_save = {"inv_items", "known_recipes"}
@@ -2039,12 +1994,14 @@ if progressive_mode then
 		end
 	end
 
-	on_leaveplayer(save_meta)
+	on_leaveplayer(function(player)
+		PLAYERS = get_players()
+		save_meta(player)
+	end)
 
 	on_shutdown(function()
-		local players = get_players()
+		for i = 1, #PLAYERS do
-		for i = 1, #players do
+			local player = PLAYERS[i]
-			local player = players[i]
 			save_meta(player)
 		end
 	end)
@@ -2072,11 +2029,13 @@ function craftguide.show(name, item, show_usages)
 	if not recipes and not usages then
 		if not recipes_cache[item] and not usages_cache[item] then
 			return false, msg(name, fmt("%s: %s",
-				S"No recipe or usage for this item", get_desc(item)))
+				S"No recipe or usage for this item",
+				get_desc(item)))
 		end
 
 		return false, msg(name, fmt("%s: %s",
-			S"You don't know a recipe or usage for this item", get_desc(item)))
+			S"You don't know a recipe or usage for this item",
+			get_desc(item)))
 	end
 
 	data.query_item = item