Default bookshelf infotext

[necron099]

Sorry, but I don't know where else to post this:
I've noticed this ever since the bookshelf has had it's own formspec. When you mouse over the bookshelf it doesn't have the "Bookshelf" text displayed on the screen like "Chest" or "Furnace" when you mouse over a chest or furnace.

The code in nodes.lua line 1576:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

on_construct = function(pos)
      local meta = minetest.get_meta(pos)
      meta:set_string("formspec", bookshelf_formspec)
      local inv = meta:get_inventory()
      inv:set_size("books", 8 * 2)
   end,

When it probably should be:

Your phone or window isn't wide enough to display the code box. If it's a phone, try rotating it to landscape mode.

Code: Select all

on_construct = function(pos)
      local meta = minetest.get_meta(pos)
      meta:set_string("formspec", bookshelf_formspec)
      meta:set_string("infotext", "Bookshelf")
      local inv = meta:get_inventory()
      inv:set_size("books", 8 * 2)
   end,

Maybe I'm just getting nit-picky in my old age but it's more uniform. Even bookshelves need to know what they are :)

[TenPlus1]

Personally I'd remove the mouseover infotext for things like chests, bookshelves etc. anything that doesn't show actual information.

[azekill_DIABLO]

yep should be optionnal

post it in feature disscussion, it's a great idea you just had ;)