trade working

* use only mouse left release in checkboxes
* remvoe duplicated code in radiogroup
* implement player trade

modules/game_playertrade/playertrade.lua

@@ -7,26 +7,55 @@ PlayerTrade = {}
 local tradeWindow
 
 local function createTrade()
-  if tradeWindow then
-    tradeWindow:destroy()
-    tradeWindow = nil
+  tradeWindow = createWidget('TradeWindow', GameInterface.getRightPanel())
+  tradeWindow.onClose = function()
+    g_game.rejectTrade()
+    tradeWindow:hide()
+  end
+end
+
+local function fillTrade(name, items, counter)
+  if not tradeWindow then
+    createTrade()
   end
 
-  tradeWindow = createWidget('TradeWindow', rootWidget)
-end
-
-local function onOwnTrade(name, items)
-  local firstItem = items[1]
-
   local tradeItemWidget = tradeWindow:getChildById('tradeItem')
-  tradeItemWidget:setItem(firstItem)
+  tradeItemWidget:setItemId(items[1]:getId())
+
+  local tradeContainer
+  local label
+  if counter then
+    tradeContainer = tradeWindow:recursiveGetChildById('counterTradeContainer')
+    label = tradeWindow:recursiveGetChildById('counterTradeLabel')
+
+    tradeWindow:recursiveGetChildById('acceptButton'):enable()
+  else
+    tradeContainer = tradeWindow:recursiveGetChildById('ownTradeContainer')
+    label = tradeWindow:recursiveGetChildById('ownTradeLabel')
+  end
+  label:setText(name)
+
+  for index,item in ipairs(items) do
+    local itemWidget = createWidget('Item', tradeContainer)
+    itemWidget:setItem(item)
+    itemWidget:setVirtual(true)
+    itemWidget:setMargin(1)
+    itemWidget.onClick = function()
+      g_game.inspectTrade(counter, index-1)
+    end
+  end
 end
 
-local function onCounterTrade(name, items)
-
+local function onGameOwnTrade(name, items)
+  fillTrade(name, items, false)
 end
 
-local function onCloseTrade()
+local function onGameCounterTrade(name, items)
+  fillTrade(name, items, true)
+end
+
+local function onGameCloseTrade()
+  if not tradeWindow then return end
   tradeWindow:destroy()
   tradeWindow = nil
 end
@@ -36,11 +65,18 @@ function PlayerTrade.init()
 
   connect(g_game, { onOwnTrade = onGameOwnTrade,
                     onCounterTrade = onGameCounterTrade,
-                    onCloseTrade = onGameCloseTrade })
+                    onCloseTrade = onGameCloseTrade,
+                    onGameEnd = onGameCloseTrade })
 end
 
 function PlayerTrade.terminate()
   disconnect(g_game, { onOwnTrade = onGameOwnTrade,
                        onCounterTrade = onGameCounterTrade,
-                       onCloseTrade = onGameCloseTrade })
+                       onCloseTrade = onGameCloseTrade,
+                       onGameEnd = onGameCloseTrade })
+
+  if tradeWindow then
+    tradeWindow:destroy()
+    tradeWindow = nil
+  end
 end

modules/game_playertrade/tradewindow.otui

@@ -15,24 +15,70 @@ TradeWindow < MiniWindow
   MiniWindowContents
     padding: 6
 
-    FlatPanel
+    ScrollableFlatPanel
       id: ownTradeContainer
-      anchors.fill: parent
-      anchors.right: parent.horizontalCenter
-      margin-right: 2
+      anchors.left: parent.left
+      anchors.top: tradeScrollBar.top
+      anchors.right: tradeScrollBar.left
+      anchors.bottom: tradeScrollBar.bottom
       layout:
         type: grid
-        cell-size: 40 40
+        cell-size: 36 36
+        flow: true
+        cell-spacing: 0
+      vertical-scrollbar: tradeScrollBar
+
+    ScrollableFlatPanel
+      id: counterTradeContainer
+      anchors.right: parent.right
+      anchors.top: tradeScrollBar.top
+      anchors.left: tradeScrollBar.right
+      anchors.bottom: tradeScrollBar.bottom
+      layout:
+        type: grid
+        cell-size: 36 36
         flow: true
         cell-spacing: 0
 
-    FlatPanel
-      id: counterTradeContainer
-      anchors.fill: parent
+    VerticalScrollBar
+      id: tradeScrollBar
+      anchors.top: parent.top
+      anchors.bottom: acceptButton.top
+      anchors.horizontalCenter: parent.horizontalCenter
+      margin-top: 14
+      margin-bottom: 4
+      step: 14
+      pixels-scroll: true
+
+    Label
+      id: ownTradeLabel
+      anchors.bottom: ownTradeContainer.top
+      anchors.left: ownTradeContainer.left
+      anchors.right: ownTradeContainer.right
+      margin-bottom: 2
+
+    Label
+      id: counterTradeLabel
+      anchors.bottom: counterTradeContainer.top
+      anchors.left: counterTradeContainer.left
+      anchors.right: counterTradeContainer.right
+      margin-bottom: 2
+
+    Button
+      !text: tr('Accept')
+      id: acceptButton
+      anchors.bottom: parent.bottom
+      anchors.left: parent.left
+      anchors.right: parent.horizontalCenter
+      margin-right: 2
+      enabled: false
+      @onClick: g_game.acceptTrade(); self:disable()
+
+    Button
+      !text: tr('Reject')
+      id: rejectButton
+      anchors.bottom: parent.bottom
+      anchors.right: parent.right
       anchors.left: parent.horizontalCenter
       margin-left: 2
-      layout:
-        type: grid
-        cell-size: 40 40
-        flow: true
-        cell-spacing: 0
+      @onClick: g_game.rejectTrade()