Redesign simple item info

gfx/prop/children.lua

@@ -122,10 +122,6 @@ function Children:with(elementType)
   end
 
   function elementType:handleEvent(evt)
-    if Element.handleEvent(self, evt) then
-      return true
-    end
-
     local evtLocalCoords = Event.toElementLocalPosition(evt, self)
 
     if Event.isClickEvent(evt) then
@@ -139,15 +135,15 @@ function Children:with(elementType)
           return true
         end
       end
-      return false
     else
       for _,child in self:_iterateChildren() do
         if child:handleEvent(evtLocalCoords) then
           return true
         end
       end
-      return false
     end
+
+    return Element.handleEvent(self, evt)
   end
 
   return elementType

itemcontroller.lua

@@ -331,11 +331,36 @@ local PAGES = {
       bgColor = colors.gray
     }
 
-    local itemCount = Text:new{
-      text = "Count: "..tostring(group:getItemCount()),
-    }
-    local itemSlots = Text:new{
-      text = "Slots: "..tostring(group:getStackCount())
+    local simpleDetailsList = List:new{
+      [Orientation:getId()] = Orientation.HORIZONTAL,
+      [Children:getId()] = {
+        Padding:new{
+          top = 0,
+          left = 1,
+          right = 1,
+          bottom = 0,
+          element = List:new{
+            [Orientation:getId()] = Orientation.VERTICAL,
+            [Children:getId()] = {
+              Text:new{ text = "Count" },
+              Text:new{ text = tostring(group:getItemCount()) }
+            }
+          }
+        },
+        Padding:new{
+          top = 0,
+          left = 1,
+          right = 1,
+          bottom = 0,
+          element = List:new{
+            [Orientation:getId()] = Orientation.VERTICAL,
+            [Children:getId()] = {
+              Text:new{ text = "Slot" },
+              Text:new{ text = tostring(group:getStackCount()) }
+            }
+          }
+        }
+      }
     }
 
     local damage = group:getDamage()
@@ -353,8 +378,13 @@ local PAGES = {
     local infoElements = List:new{
       [Children:getId()] = {
         paddedTitle,
-        itemCount,
-        itemSlots,
+        Padding:new{
+          top = 0,
+          bottom = 1,
+          left = 0,
+          right = 0,
+          element = simpleDetailsList
+        },
         damageBar
       },
       [Orientation:getId()] = Orientation.VERTICAL,