diff --git a/Lib/tkinter/simpledialog.py b/Lib/tkinter/simpledialog.py index 96b8b90d215..885804b3c70 100644 --- a/Lib/tkinter/simpledialog.py +++ b/Lib/tkinter/simpledialog.py @@ -130,6 +130,7 @@ def __init__(self, parent, title = None): ''' Toplevel.__init__(self, parent) + self.withdraw() # remain invisible for now # If the master is not viewable, don't # make the child transient, or else it # would be opened withdrawn @@ -149,9 +150,6 @@ def __init__(self, parent, title = None): self.buttonbox() - self.wait_visibility() # window needs to be visible for the grab - self.grab_set() - if not self.initial_focus: self.initial_focus = self @@ -161,8 +159,13 @@ def __init__(self, parent, title = None): self.geometry("+%d+%d" % (parent.winfo_rootx()+50, parent.winfo_rooty()+50)) + self.deiconify() # become visible now + self.initial_focus.focus_set() + # wait for window to appear on screen before calling grab_set + self.wait_visibility() + self.grab_set() self.wait_window(self) def destroy(self): diff --git a/Misc/NEWS b/Misc/NEWS index 45bd799804c..48cb004a031 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -183,6 +183,10 @@ Core and Builtins Library ------- +- Issue #2638: Show a window constructed with tkSimpleDialog.Dialog only after + it is has been populated and properly configured in order to prevent + window flashing. + - Issue #4792: Prevent a segfault in _tkinter by using the guaranteed to be safe interp argument given to the PythonCmd in place of the Tcl interpreter taken from a PythonCmd_ClientData.