Commit 33c7420e authored by Terry Jan Reedy's avatar Terry Jan Reedy Committed by GitHub

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').
parent a361e89d
...@@ -246,7 +246,7 @@ class AutoCompleteWindow: ...@@ -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: ...@@ -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)
......
...@@ -267,7 +267,7 @@ class EditorWindow(object): ...@@ -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): ...@@ -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): ...@@ -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
......
...@@ -10,7 +10,7 @@ class MainMenuTest(unittest.TestCase): ...@@ -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):
......
...@@ -148,7 +148,7 @@ def overrideRootMenu(root, flist): ...@@ -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):
......
...@@ -97,7 +97,7 @@ menudefs = [ ...@@ -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>>'),
]), ]),
......
...@@ -852,7 +852,7 @@ class PyShell(OutputWindow): ...@@ -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"),
] ]
......
Change mainmenu.menudefs key 'windows' to 'window'. Every other menudef key
is lowercase version of main menu entry.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment