Page 1 of 1

Default bookshelf infotext

PostPosted: Wed May 18, 2016 15:49
by 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 :)

Re: Default bookshelf infotext

PostPosted: Wed May 18, 2016 18:12
by TenPlus1
Personally I'd remove the mouseover infotext for things like chests, bookshelves etc. anything that doesn't show actual information.

Re: Default bookshelf infotext

PostPosted: Thu May 19, 2016 15:43
by azekill_DIABLO
yep should be optionnal

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