bpo-33924: Change IDLE mainmenu.menudefs key 'windows' to 'window' (GH-7836)

Every other menudef key is the lowercase version of the
corresponding main menu entry (in this case, 'Window').
(cherry picked from commit 33c7420e7d)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
This commit is contained in:
Miss Islington (bot) 2018-06-20 21:35:47 -07:00 committed by GitHub
parent f9243a22da
commit e1f0dceb26
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 11 additions and 9 deletions

View File

@ -246,7 +246,7 @@ class AutoCompleteWindow:
acw.wm_geometry("+%d+%d" % (new_x, new_y)) acw.wm_geometry("+%d+%d" % (new_x, new_y))
if platform.system().startswith('Windows'): if platform.system().startswith('Windows'):
# See issue 15786. When on windows platform, Tk will misbehave # See issue 15786. When on Windows platform, Tk will misbehave
# to call winconfig_event multiple times, we need to prevent this, # to call winconfig_event multiple times, we need to prevent this,
# otherwise mouse button double click will not be able to used. # otherwise mouse button double click will not be able to used.
acw.unbind(WINCONFIG_SEQUENCE, self.winconfigid) acw.unbind(WINCONFIG_SEQUENCE, self.winconfigid)
@ -269,7 +269,7 @@ class AutoCompleteWindow:
# mouse click on widget / text area. # mouse click on widget / text area.
if self.is_active(): if self.is_active():
if event.type == EventType.FocusOut: if event.type == EventType.FocusOut:
# On windows platform, it will need to delay the check for # On Windows platform, it will need to delay the check for
# acw.focus_get() when click on acw, otherwise it will return # acw.focus_get() when click on acw, otherwise it will return
# None and close the window # None and close the window
self.widget.after(1, self._hide_event_check) self.widget.after(1, self._hide_event_check)

View File

@ -267,7 +267,7 @@ class EditorWindow(object):
self.saved_change_hook() self.saved_change_hook()
self.update_recent_files_list() self.update_recent_files_list()
self.load_extensions() self.load_extensions()
menu = self.menudict.get('windows') menu = self.menudict.get('window')
if menu: if menu:
end = menu.index("end") end = menu.index("end")
if end is None: if end is None:
@ -410,7 +410,7 @@ class EditorWindow(object):
("format", "F_ormat"), ("format", "F_ormat"),
("run", "_Run"), ("run", "_Run"),
("options", "_Options"), ("options", "_Options"),
("windows", "_Window"), ("window", "_Window"),
("help", "_Help"), ("help", "_Help"),
] ]
@ -437,7 +437,7 @@ class EditorWindow(object):
def postwindowsmenu(self): def postwindowsmenu(self):
# Only called when Window menu exists # Only called when Window menu exists
menu = self.menudict['windows'] menu = self.menudict['window']
end = menu.index("end") end = menu.index("end")
if end is None: if end is None:
end = -1 end = -1

View File

@ -10,7 +10,7 @@ class MainMenuTest(unittest.TestCase):
def test_menudefs(self): def test_menudefs(self):
actual = [item[0] for item in mainmenu.menudefs] actual = [item[0] for item in mainmenu.menudefs]
expect = ['file', 'edit', 'format', 'run', 'shell', expect = ['file', 'edit', 'format', 'run', 'shell',
'debug', 'options', 'windows', 'help'] 'debug', 'options', 'window', 'help']
self.assertEqual(actual, expect) self.assertEqual(actual, expect)
def test_default_keydefs(self): def test_default_keydefs(self):

View File

@ -148,7 +148,7 @@ def overrideRootMenu(root, flist):
root.configure(menu=menubar) root.configure(menu=menubar)
menudict = {} menudict = {}
menudict['windows'] = menu = Menu(menubar, name='windows', tearoff=0) menudict['window'] = menu = Menu(menubar, name='window', tearoff=0)
menubar.add_cascade(label='Window', menu=menu, underline=0) menubar.add_cascade(label='Window', menu=menu, underline=0)
def postwindowsmenu(menu=menu): def postwindowsmenu(menu=menu):

View File

@ -97,7 +97,7 @@ menudefs = [
('_Code Context', '<<toggle-code-context>>'), ('_Code Context', '<<toggle-code-context>>'),
]), ]),
('windows', [ ('window', [
('Zoom Height', '<<zoom-height>>'), ('Zoom Height', '<<zoom-height>>'),
]), ]),

View File

@ -852,7 +852,7 @@ class PyShell(OutputWindow):
("edit", "_Edit"), ("edit", "_Edit"),
("debug", "_Debug"), ("debug", "_Debug"),
("options", "_Options"), ("options", "_Options"),
("windows", "_Window"), ("window", "_Window"),
("help", "_Help"), ("help", "_Help"),
] ]

View File

@ -0,0 +1,2 @@
Change mainmenu.menudefs key 'windows' to 'window'. Every other menudef key
is lowercase version of main menu entry.