From 0135e386c264e0c6b50cb328d38547d16799cb77 Mon Sep 17 00:00:00 2001
From: lufebe16 <lufebe16@bluewin.ch>
Date: Tue, 12 Nov 2024 10:47:35 +0100
Subject: [PATCH] Android/Kivy: - Event handling: Pass through for empty
 stacks. - Menu entry to reset zoom.

---
 pysollib/kivy/LApp.py     | 12 ++++++++----
 pysollib/kivy/menubar.py  |  6 ++++++
 pysollib/kivy/tkwidget.py |  5 +++++
 3 files changed, 19 insertions(+), 4 deletions(-)

diff --git a/pysollib/kivy/LApp.py b/pysollib/kivy/LApp.py
index dcd350d7..d5965732 100644
--- a/pysollib/kivy/LApp.py
+++ b/pysollib/kivy/LApp.py
@@ -806,6 +806,10 @@ class LImageItem(BoxLayout, LBase):
     - Most events return EVENT_HANDLED even if they did not change anything
       in current situations. I would expect specifically for stack base cards
       that they return HANDLE_PROPAGATE if nothing happened.
+    - stack __defaultclickhandler__ returns EVENT_HANDLED in any case so some
+      code here is obsolete or for future.
+    - A pragmatic way to handle this: If an empty stack is still empty
+      after the click then we propagate otherwise not.
     LB241111.
     '''
 
@@ -851,7 +855,6 @@ class LImageItem(BoxLayout, LBase):
                             self.dragstart = touch.pos
                             event.cardid = i
                             r = self.send_event_pressed(touch, event)
-                            # print("********* event return = ",r)
                             if r == EVENT_HANDLED:
                                 AndroidScreenRotation.lock(toaster=False)
                                 print('grab')
@@ -869,7 +872,8 @@ class LImageItem(BoxLayout, LBase):
                     event.y = ppos[1]
                     r = self.group.bindings['<1>'](event)
                     if r == EVENT_HANDLED:
-                        return True
+                        if len(self.group.stack.cards) > 0:
+                            return True
                     return False
 
             if self.card is None:
@@ -907,7 +911,6 @@ class LImageItem(BoxLayout, LBase):
                             event.y = ppos[1]
                             event.cardid = i
                             r = self.send_event_released_1(event)
-                            # print("********* event return = ",r)
                             if r == EVENT_HANDLED:
                                 return True
                             return False
@@ -922,7 +925,8 @@ class LImageItem(BoxLayout, LBase):
                     event.y = ppos[1]
                     r = self.group.bindings['<ButtonRelease-1>'](event)
                     if r == EVENT_HANDLED:
-                        return True
+                        if len(self.group.stack.cards) > 0:
+                            return True
                     return False
 
             if self.card is None:
diff --git a/pysollib/kivy/menubar.py b/pysollib/kivy/menubar.py
index c6521173..4734197e 100644
--- a/pysollib/kivy/menubar.py
+++ b/pysollib/kivy/menubar.py
@@ -387,6 +387,9 @@ class EditMenuDialog(LMenuDialog):  # Tools
             text=_('Shuffle tiles'), command=self.menubar.mShuffle))
         tv.add_node(LTreeNode(
             text=_('Deal cards'), command=self.menubar.mDeal))
+        tv.add_node(LTreeNode(
+            text=_('Reset zoom'),
+            command=self.auto_close(self.menubar.mResetZoom)))
 
         self.addCheckNode(tv, None,
                           _('Pause'),
@@ -2141,6 +2144,9 @@ class PysolMenubarTk:
                 toast.show(parent=baseWindow, duration=5.0)
             self.updateMenus()
 
+    def mResetZoom(self, *args):
+        self.tkopt.table_zoom.value = [1.0, 0.0, 0.0]
+
     def mPause(self, *args):
         if not self.game:
             return
diff --git a/pysollib/kivy/tkwidget.py b/pysollib/kivy/tkwidget.py
index bc053f83..cecc33c2 100644
--- a/pysollib/kivy/tkwidget.py
+++ b/pysollib/kivy/tkwidget.py
@@ -397,6 +397,10 @@ class LScatterFrame(Scatter):
         yoff = zoom[2]
         self.offset = (xoff,yoff)
 
+    def _change_command(self,inst,val):
+        if self.lock_pos is None:
+            self.set_scale(val)
+
     def _update(self):
         # initialisation
         if self.tkopt is None:
@@ -407,6 +411,7 @@ class LScatterFrame(Scatter):
                 self.tkopt = tkopt
                 self.set_scale(tkopt.table_zoom.value)
                 print("table_zoom",tkopt.table_zoom.value)
+                tkopt.table_zoom.bind(value=self._change_command)
 
         # update
         if self.lock_pos is None: