Modify copy_widget to allow non-visible widgets

Author: RedFantom

tests/test_utilities.py

@@ -73,6 +73,17 @@ def test_move_widget_place(self):
         self.assertNotIn(label, self.window.place_slaves())
         self.assertGeometryInfoEquals(info, label.place_info())
 
+    def test_move_widget_none(self):
+        label = ttk.Label(self.window)
+        self.assertFalse(label.place_info() is True)
+        self.assertFalse(label.grid_info() is True)
+        self.assertRaises(tk.TclError, label.pack_info)
+        tl = tk.Toplevel(self.window)
+        label = move_widget(label, tl, preserve_geometry=True)
+        self.assertFalse(label.place_info() is True)
+        self.assertFalse(label.grid_info() is True)
+        self.assertRaises(tk.TclError, label.pack_info)
+
     def test_move_widget_with_binding(self):
         label = ttk.Label(self.window)
         label.bind('<Enter>', self._dummy_bind)

ttkwidgets/utilities.py

@@ -8,6 +8,10 @@
 import os
 from PIL import Image, ImageTk
 import re
+try:
+    import Tkinter as tk
+except ImportError:
+    import tkinter as tk
 
 
 def get_assets_directory():
@@ -52,6 +56,10 @@ def copy_widget(widget, new_parent, level=0):
         script = widget.bind(b)
         rv.bind(b, script)
     if level > 0:
+        try:
+            pack_info = widget.pack_info()
+        except tk.TclError:
+            pack_info = {}
         if widget.grid_info():  # if geometry manager is grid
             temp = widget.grid_info()
             del temp['in']
@@ -60,10 +68,12 @@ def copy_widget(widget, new_parent, level=0):
             temp = widget.place_info()
             del temp['in']
             rv.place(**temp)
-        else:  # if geometry manager is pack
+        elif pack_info:  # if geometry manager is pack
             temp = widget.pack_info()
             del temp['in']
             rv.pack(**temp)
+        else:  # No geometry manager configured
+            pass
     level += 1
 
     if widget.pack_slaves():  # subwidgets are using the pack() geometry manager