Merge 7c354fb into 61000de

seanbudd · web-flow · commit 388be3416f09 · 2022-02-08T16:22:57.000+11:00
diff --git a/source/gui/settingsDialogs.py b/source/gui/settingsDialogs.py
@@ -274,9 +274,17 @@ def onApply(self, evt):
 		self.postInit()
 		self.SetReturnCode(wx.ID_APPLY)
 
-	def _onWindowDestroy(self, evt):
-		evt.Skip()
-		self._setInstanceDestroyedState()
+	def _onWindowDestroy(self, evt: wx.WindowDestroyEvent):
+		# Destroy events are sent to parent windows for handling.
+		# If a child window is being destroyed, we don't want to
+		# set this object as destroyed.
+		# The ExpandoTextCtrl creates a destroy event as part of
+		# initialization, this caused the NVDASettings dialog
+		# to be incorrectly set to destroyed, causing #12818.
+		isSelfAlive = bool(self)
+		if not isSelfAlive:
+			evt.Skip()
+			self._setInstanceDestroyedState()
 
 # An event and event binder that will notify the containers that they should
 # redo the layout in whatever way makes sense for their particular content.
diff --git a/tests/system/nvdaSettingsFiles/settings-gestures.ini b/tests/system/nvdaSettingsFiles/settings-gestures.ini
@@ -0,0 +1,2 @@
+[globalCommands.GlobalCommands]
+activateBrailleSettingsDialog = kb:control+/

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+[globalCommands.GlobalCommands]`
	`2`	`+activateBrailleSettingsDialog = kb:control+/`