abriflame should have a recipe in the inventory #2461

New Issue

alphabet probably should also show up in unified_inventory, i think that's just the matter of removing the not_in_creative_inventory group. is there any reason for that currently?

> Jonez, alphabet, abriflame, ... my latest PR to jonez (already accepted) adds recipes for those nodes https://github.com/runsy/jonez/pull/6 alphabet probably should also show up in unified_inventory, i think that's just the matter of removing the `not_in_creative_inventory` group. is there any reason for that currently?

flux added the

2. prio/interesting

label 2022-08-16 01:38:30 +00:00

flux added the

4. step/question

4. step/want approval

labels 2022-08-16 02:39:27 +00:00

AliasAlreadyTaken commented

2022-08-20 05:27:55 +00:00

There's a reason for alphabet not being in the inv, as well as some others: To not clog up the inv with easily guessable blocks.

If there's an A, there must be a B. Same could be said for others though, especially the non-sawable 16 colour glass/wood/stone/... or the non-craftables like beacons or rainbow water.

Since The inventory requires a really giant string which causes lag during login I'd like to have as many things out of the inventory as possible.

Unfortunately people with creative do not have a "different" view: https://github.com/minetest-mods/unified_inventory/issues/178

There's a reason for alphabet not being in the inv, as well as some others: To not clog up the inv with easily guessable blocks. If there's an A, there must be a B. Same could be said for others though, especially the non-sawable 16 colour glass/wood/stone/... or the non-craftables like beacons or rainbow water. Since The inventory requires a really giant string which causes lag during login I'd like to have as many things out of the inventory as possible. Unfortunately people with creative do not have a "different" view: https://github.com/minetest-mods/unified_inventory/issues/178

flux commented

2022-08-20 16:52:28 +00:00

There's a reason for alphabet not being in the inv, as well as some others: To not clog up the inv with easily guessable blocks.

If there's an A, there must be a B. Same could be said for others though, especially the non-sawable 16 colour glass/wood/stone/... or the non-craftables like beacons or rainbow water.

i'm talking about these:

whose recipes are only visible if you look at the source code: https://gitea.your-land.de/your-land/alphabet/src/branch/master/init.lua
and are recipes like

minetest.register_craft({
	output = 'alphabet:d 1',
	recipe = {
		{'default:dirt', 'default:dirt', ''},
		{'', 'default:dirt', 'default:wood'}
	}
})

how is that easily guessable?

Since The inventory requires a really giant string which causes lag during login I'd like to have as many things out of the inventory as possible.

reasonable, though i feel like this needs to be fixed by refactoring the unified_inventory GUI so as to not require so much processing, at least when a player first logs in.

Unfortunately people with creative do not have a "different" view: https://github.com/minetest-mods/unified_inventory/issues/178

> There's a reason for alphabet not being in the inv, as well as some others: To not clog up the inv with easily guessable blocks. > > If there's an A, there must be a B. Same could be said for others though, especially the non-sawable 16 colour glass/wood/stone/... or the non-craftables like beacons or rainbow water. i'm talking about these: ![](https://gitea.your-land.de/attachments/2bb8ab50-bbc2-4dc4-be5b-656dfba9825c) whose recipes are only visible if you look at the source code: https://gitea.your-land.de/your-land/alphabet/src/branch/master/init.lua and are recipes like ```lua minetest.register_craft({ output = 'alphabet:d 1', recipe = { {'default:dirt', 'default:dirt', ''}, {'', 'default:dirt', 'default:wood'} } }) ``` how is that easily guessable? > Since The inventory requires a really giant string which causes lag during login I'd like to have as many things out of the inventory as possible. reasonable, though i feel like this needs to be fixed by refactoring the unified_inventory GUI so as to not require so much processing, at least when a player first logs in. > Unfortunately people with creative do not have a "different" view: https://github.com/minetest-mods/unified_inventory/issues/178

screenshot_20220820_094719.png

600 KiB

AliasAlreadyTaken commented

2022-08-20 21:14:14 +00:00

how is that easily guessable?

The recipes are not at all easily guessable, to be honest, even if I knew five or ten of those recipes, I'd certainly not think "oh, sure, braille" immediately and be able to guess the others. Unless one knows they exist and where to look. That's what we need to hint at, among the others.

We could now either

build instructions in the advanced tutorial
add the alphabet blocks to the visible inventory
create a machine (like a printing press) that makes alphabet blocks from colour and clay(?)
fix unified inventory

> how is that easily guessable? The recipes are not at all easily guessable, to be honest, even if I knew five or ten of those recipes, I'd certainly not think "oh, sure, braille" immediately and be able to guess the others. Unless one knows they exist and where to look. That's what we need to hint at, among the others. We could now either - build instructions in the advanced tutorial - add the alphabet blocks to the visible inventory - create a machine (like a printing press) that makes alphabet blocks from colour and clay(?) - fix unified inventory

flux commented

2022-08-21 17:37:36 +00:00

"oh, sure, braille" immediately and be able to guess the others

if it was braille w/ just 1 crafting material, that'd be one thing. but it's also got the 2nd material in the 3rd column.

build instructions in the advanced tutorial

add the alphabet blocks to the visible inventory

create a machine (like a printing press) that makes alphabet blocks from colour and clay(?)

fix unified inventory

i think right now, we should just make them visible in the inventory. sometime soon, we can add a letter press (i suggest taking the node model/textures from the letter press in https://github.com/minetest-mods/letters). eventually i want to work on fixing unified inventory (or perhaps creating an entirely new inventory manager), but that's not gonna happen anytime soon.

> "oh, sure, braille" immediately and be able to guess the others if it was braille w/ just 1 crafting material, that'd be one thing. but it's also got the 2nd material in the 3rd column. > - build instructions in the advanced tutorial > - add the alphabet blocks to the visible inventory > - create a machine (like a printing press) that makes alphabet blocks from colour and clay(?) > - fix unified inventory i think right now, we should just make them visible in the inventory. sometime soon, we can add a letter press (i suggest taking the node model/textures from the letter press in https://github.com/minetest-mods/letters). eventually i want to work on fixing unified inventory (or perhaps creating an entirely new inventory manager), but that's not gonna happen anytime soon.

flux commented

2022-08-22 00:35:48 +00:00

abriflame "recipes" are now in unified_inventory w/ c5a2e638fa

abriflame "recipes" are now in unified_inventory w/ https://gitea.your-land.de/your-land/yl_commons/commit/c5a2e638fae9b0e23c8b32c36a6a81250a5fb376

👍 1

flux added the

4. step/ready to QA test

label 2022-08-22 00:36:02 +00:00

AliasAlreadyTaken was assigned by flux

2022-08-22 00:37:03 +00:00

AliasAlreadyTaken added this to the 1.1.115 milestone 2022-08-22 00:39:24 +00:00

flux added