diff options
| author | Miss Skeleton (bot) <31488909+miss-islington@users.noreply.github.com> | 2020-10-08 16:50:36 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-10-08 16:50:36 (GMT) |
| commit | 15e091f63f12b659ad17563a4ce7f5188aceae0a (patch) | |
| tree | bacdbcd8737a22fb17ec3f7b36b461cba8afb447 /Lib/tkinter | |
| parent | 5b1fdcacfc36dfe36693f4c350c958df6600de14 (diff) | |
| download | cpython-15e091f63f12b659ad17563a4ce7f5188aceae0a.zip cpython-15e091f63f12b659ad17563a4ce7f5188aceae0a.tar.gz cpython-15e091f63f12b659ad17563a4ce7f5188aceae0a.tar.bz2 | |
bpo-41306: Allow scale value to not be rounded (GH-21715)
This fixes the test failure with Tk 6.8.10 which is caused by changes to how Tk rounds the `from`, `to` and `tickinterval` arguments. This PR uses `noconv` if the patchlevel is greater than or equal to 8.6.10 (credit to Serhiy for this idea as it is much simpler than what I previously proposed).
Going into more detail for those who want it, the Tk change was made in [commit 591f68c](https://github.com/tcltk/tk/commit/591f68cb382525b72664c6fecaab87742b6cc87a) and means that the arguments listed above are rounded relative to the value of `from`. However, when rounding the `from` argument ([line 623](https://github.com/tcltk/tk/blob/591f68cb382525b72664c6fecaab87742b6cc87a/generic/tkScale.cGH-L623)), it is rounded relative to itself (i.e. rounding `0`) and therefore the assigned value for `from` is always what is given (no matter what values of `from` and `resolution`).
Automerge-Triggered-By: @pablogsal
(cherry picked from commit aecf036738a404371303e770f4ce4fd9f7d43de7)
Co-authored-by: E-Paine <63801254+E-Paine@users.noreply.github.com>
Diffstat (limited to 'Lib/tkinter')
| -rw-r--r-- | Lib/tkinter/test/test_tkinter/test_widgets.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Lib/tkinter/test/test_tkinter/test_widgets.py b/Lib/tkinter/test/test_tkinter/test_widgets.py index 721e813..b6f792d 100644 --- a/Lib/tkinter/test/test_tkinter/test_widgets.py +++ b/Lib/tkinter/test/test_tkinter/test_widgets.py @@ -940,7 +940,8 @@ class ScaleTest(AbstractWidgetTest, unittest.TestCase): def test_from(self): widget = self.create() - self.checkFloatParam(widget, 'from', 100, 14.9, 15.1, conv=float_round) + conv = False if get_tk_patchlevel() >= (8, 6, 10) else float_round + self.checkFloatParam(widget, 'from', 100, 14.9, 15.1, conv=conv) def test_label(self): widget = self.create() |