josephsl/wintenApps

Global plugin: remove additional events handler

Closed this issue · 1 comments

Hi,

To be implemented when additional events and properties become empty for some ime:

In the past, when events such as drag and drop were not part of NVDA Core, Windows App Essentials added support for them by registering a global event handler to take care of these events. When NVDA Core added these events and properties, they were removed from the add-on but the global event handler group was still created.

As of NVDA 2022.4 (and subsequently with add-on code based on it), there are no more additional events and properties supported by the add-on as these recorded drag and drop events. Therefore once a stable version of the add-on requiring NVDA 2022.4 is released, consider removing additional events and properties support from the add-on with rooms for its return if events and properties must be supported again. But before removing event registration routine, take care of some optimizations such as not entering this routine of events and properties maps are empty.

Removing additional events and properties support reduces the scope of Windows App Essentials significantly. But with more features from the add-on being integrated into NVDA Core and since Windows 10 is in maintenance mode as demonstrated by Version 22H2 (2022 Update) release, it makes more sense to remove additional events and properties registration routine. But if the need arises to bring support for more events and properties, this routine can be resurrected.

Thanks.

Hi,

This change will be part of the same release as the stable version requiring NVDA 2022.4 for consistency.

Thanks.