bpo-33906: Rename idlelib.windows as window (GH-7833)
Match Window on the main menu and remove last plural module name.
Change imports, test, and attribute references to match new name.
(cherry picked from commit a361e89d5a
)
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
This commit is contained in:
parent
7c853d0620
commit
2839da4e51
|
@ -16,7 +16,7 @@ import sys
|
|||
from idlelib.config import idleConf
|
||||
from idlelib import pyshell
|
||||
from idlelib.tree import TreeNode, TreeItem, ScrolledCanvas
|
||||
from idlelib.windows import ListedToplevel
|
||||
from idlelib.window import ListedToplevel
|
||||
|
||||
|
||||
file_open = None # Method...Item and Class...Item use this.
|
||||
|
|
|
@ -6,7 +6,7 @@ from tkinter.ttk import Scrollbar
|
|||
|
||||
from idlelib import macosx
|
||||
from idlelib.scrolledlist import ScrolledList
|
||||
from idlelib.windows import ListedToplevel
|
||||
from idlelib.window import ListedToplevel
|
||||
|
||||
|
||||
class Idb(bdb.Bdb):
|
||||
|
@ -40,7 +40,7 @@ class Idb(bdb.Bdb):
|
|||
prev_name = prev_frame.f_code.co_filename
|
||||
if 'idlelib' in prev_name and 'debugger' in prev_name:
|
||||
# catch both idlelib/debugger.py and idlelib/debugger_r.py
|
||||
# on both posix and windows
|
||||
# on both Posix and Windows
|
||||
return False
|
||||
return self.in_rpc_code(prev_frame)
|
||||
|
||||
|
|
|
@ -25,7 +25,7 @@ from idlelib import pyparse
|
|||
from idlelib import query
|
||||
from idlelib import replace
|
||||
from idlelib import search
|
||||
from idlelib import windows
|
||||
from idlelib import window
|
||||
|
||||
# The default tab setting for a Text widget, in average-width characters.
|
||||
TK_TABWIDTH_DEFAULT = 8
|
||||
|
@ -101,7 +101,7 @@ class EditorWindow(object):
|
|||
root = root or flist.root
|
||||
self.root = root
|
||||
self.menubar = Menu(root)
|
||||
self.top = top = windows.ListedToplevel(root, menu=self.menubar)
|
||||
self.top = top = window.ListedToplevel(root, menu=self.menubar)
|
||||
if flist:
|
||||
self.tkinter_vars = flist.vars
|
||||
#self.top.instance_dict makes flist.inversedict available to
|
||||
|
@ -138,7 +138,7 @@ class EditorWindow(object):
|
|||
self.top.protocol("WM_DELETE_WINDOW", self.close)
|
||||
self.top.bind("<<close-window>>", self.close_event)
|
||||
if macosx.isAquaTk():
|
||||
# Command-W on editorwindows doesn't work without this.
|
||||
# Command-W on editor windows doesn't work without this.
|
||||
text.bind('<<close-window>>', self.close_event)
|
||||
# Some OS X systems have only one mouse button, so use
|
||||
# control-click for popup context menus there. For two
|
||||
|
@ -276,7 +276,7 @@ class EditorWindow(object):
|
|||
menu.add_separator()
|
||||
end = end + 1
|
||||
self.wmenu_end = end
|
||||
windows.register_callback(self.postwindowsmenu)
|
||||
window.register_callback(self.postwindowsmenu)
|
||||
|
||||
# Some abstractions so IDLE extensions are cross-IDE
|
||||
self.askyesno = tkMessageBox.askyesno
|
||||
|
@ -436,14 +436,14 @@ class EditorWindow(object):
|
|||
self.reset_help_menu_entries()
|
||||
|
||||
def postwindowsmenu(self):
|
||||
# Only called when Windows menu exists
|
||||
# Only called when Window menu exists
|
||||
menu = self.menudict['windows']
|
||||
end = menu.index("end")
|
||||
if end is None:
|
||||
end = -1
|
||||
if end > self.wmenu_end:
|
||||
menu.delete(self.wmenu_end+1, end)
|
||||
windows.add_windows_to_menu(menu)
|
||||
window.add_windows_to_menu(menu)
|
||||
|
||||
def handle_yview(self, event, *args):
|
||||
"Handle scrollbar."
|
||||
|
@ -1012,7 +1012,7 @@ class EditorWindow(object):
|
|||
def _close(self):
|
||||
if self.io.filename:
|
||||
self.update_recent_files_list(new_file=self.io.filename)
|
||||
windows.unregister_callback(self.postwindowsmenu)
|
||||
window.unregister_callback(self.postwindowsmenu)
|
||||
self.unload_extensions()
|
||||
self.io.close()
|
||||
self.io = None
|
||||
|
|
|
@ -255,9 +255,9 @@ class IdleConfTest(unittest.TestCase):
|
|||
with self.assertRaises(FileNotFoundError):
|
||||
conf.GetUserCfgDir()
|
||||
|
||||
@unittest.skipIf(not sys.platform.startswith('win'), 'this is test for windows system')
|
||||
@unittest.skipIf(not sys.platform.startswith('win'), 'this is test for Windows system')
|
||||
def test_get_user_cfg_dir_windows(self):
|
||||
"Test to get user config directory under windows"
|
||||
"Test to get user config directory under Windows"
|
||||
conf = self.new_config(_utest=True)
|
||||
|
||||
# Check normal way should success
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
"Test windows, coverage 47%."
|
||||
"Test window, coverage 47%."
|
||||
|
||||
from idlelib import windows
|
||||
from idlelib import window
|
||||
import unittest
|
||||
from test.support import requires
|
||||
from tkinter import Tk
|
||||
|
@ -9,7 +9,7 @@ from tkinter import Tk
|
|||
class WindowListTest(unittest.TestCase):
|
||||
|
||||
def test_init(self):
|
||||
wl = windows.WindowList()
|
||||
wl = window.WindowList()
|
||||
self.assertEqual(wl.dict, {})
|
||||
self.assertEqual(wl.callbacks, [])
|
||||
|
||||
|
@ -20,14 +20,14 @@ class ListedToplevelTest(unittest.TestCase):
|
|||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
windows.registry = set()
|
||||
window.registry = set()
|
||||
requires('gui')
|
||||
cls.root = Tk()
|
||||
cls.root.withdraw()
|
||||
|
||||
@classmethod
|
||||
def tearDownClass(cls):
|
||||
windows.registry = windows.WindowList()
|
||||
window.registry = window.WindowList()
|
||||
cls.root.update_idletasks()
|
||||
## for id in cls.root.tk.call('after', 'info'):
|
||||
## cls.root.after_cancel(id) # Need for EditorWindow.
|
||||
|
@ -36,8 +36,8 @@ class ListedToplevelTest(unittest.TestCase):
|
|||
|
||||
def test_init(self):
|
||||
|
||||
win = windows.ListedToplevel(self.root)
|
||||
self.assertIn(win, windows.registry)
|
||||
win = window.ListedToplevel(self.root)
|
||||
self.assertIn(win, window.registry)
|
||||
self.assertEqual(win.focused_widget, win)
|
||||
|
||||
|
|
@ -128,7 +128,7 @@ def overrideRootMenu(root, flist):
|
|||
# menu.
|
||||
from tkinter import Menu
|
||||
from idlelib import mainmenu
|
||||
from idlelib import windows
|
||||
from idlelib import window
|
||||
|
||||
closeItem = mainmenu.menudefs[0][1][-2]
|
||||
|
||||
|
|
|
@ -87,11 +87,11 @@ class ListedToplevel(Toplevel):
|
|||
self.tkraise()
|
||||
self.focused_widget.focus_set()
|
||||
except TclError:
|
||||
# This can happen when the window menu was torn off.
|
||||
# This can happen when the Window menu was torn off.
|
||||
# Simply ignore it.
|
||||
pass
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
from unittest import main
|
||||
main('idlelib.idle_test.test_windows', verbosity=2)
|
||||
main('idlelib.idle_test.test_window', verbosity=2)
|
|
@ -0,0 +1,2 @@
|
|||
Rename idlelib.windows as window Match Window on the main menu and remove
|
||||
last plural module name.
|
Loading…
Reference in New Issue