summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Doc/whatsnew/3.12.rst11
-rw-r--r--Lib/test/test_tkinter/test_widgets.py6
-rw-r--r--Lib/tkinter/__init__.py2
-rw-r--r--Misc/NEWS.d/next/Library/2022-10-20-14-03-58.gh-issue-94473.pzGX73.rst3
4 files changed, 21 insertions, 1 deletions
diff --git a/Doc/whatsnew/3.12.rst b/Doc/whatsnew/3.12.rst
index caf2107..efbd2ca 100644
--- a/Doc/whatsnew/3.12.rst
+++ b/Doc/whatsnew/3.12.rst
@@ -532,6 +532,17 @@ threading
profiling functions in all running threads in addition to the calling one.
(Contributed by Pablo Galindo in :gh:`93503`.)
+tkinter
+-------
+
+* ``tkinter.Canvas.coords()`` now flattens its arguments.
+ It now accepts not only coordinates as separate arguments
+ (``x1, y1, x2, y2, ...``) and a sequence of coordinates
+ (``[x1, y1, x2, y2, ...]``), but also coordinates grouped in pairs
+ (``(x1, y1), (x2, y2), ...`` and ``[(x1, y1), (x2, y2), ...]``),
+ like ``create_*()`` methods.
+ (Contributed by Serhiy Storchaka in :gh:`94473`.)
+
types
-----
diff --git a/Lib/test/test_tkinter/test_widgets.py b/Lib/test/test_tkinter/test_widgets.py
index b452451..76cc16e 100644
--- a/Lib/test/test_tkinter/test_widgets.py
+++ b/Lib/test/test_tkinter/test_widgets.py
@@ -901,6 +901,12 @@ class CanvasTest(AbstractWidgetTest, unittest.TestCase):
c.coords(i, [21, 31, 41, 51, 61, 11])
self.assertEqual(c.coords(i), [21.0, 31.0, 41.0, 51.0, 61.0, 11.0])
+ c.coords(i, (22, 32), (42, 52), (62, 12))
+ self.assertEqual(c.coords(i), [22.0, 32.0, 42.0, 52.0, 62.0, 12.0])
+
+ c.coords(i, [(23, 33), (43, 53), (63, 13)])
+ self.assertEqual(c.coords(i), [23.0, 33.0, 43.0, 53.0, 63.0, 13.0])
+
c.coords(i, 20, 30, 60, 10)
self.assertEqual(c.coords(i), [20.0, 30.0, 60.0, 10.0])
self.assertEqual(c.bbox(i), (18, 8, 62, 32))
diff --git a/Lib/tkinter/__init__.py b/Lib/tkinter/__init__.py
index bf0b3b9..c675c51 100644
--- a/Lib/tkinter/__init__.py
+++ b/Lib/tkinter/__init__.py
@@ -2817,7 +2817,7 @@ class Canvas(Widget, XView, YView):
def coords(self, *args):
"""Return a list of coordinates for the item given in ARGS."""
- # XXX Should use _flatten on args
+ args = _flatten(args)
return [self.tk.getdouble(x) for x in
self.tk.splitlist(
self.tk.call((self._w, 'coords') + args))]
diff --git a/Misc/NEWS.d/next/Library/2022-10-20-14-03-58.gh-issue-94473.pzGX73.rst b/Misc/NEWS.d/next/Library/2022-10-20-14-03-58.gh-issue-94473.pzGX73.rst
new file mode 100644
index 0000000..f70722a
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2022-10-20-14-03-58.gh-issue-94473.pzGX73.rst
@@ -0,0 +1,3 @@
+Flatten arguments in :meth:`tkinter.Canvas.coords`. It now accepts not only
+``x1, y1, x2, y2, ...`` and ``[x1, y1, x2, y2, ...]``, but also ``(x1, y1),
+(x2, y2), ...`` and ``[(x1, y1), (x2, y2), ...]``.