Repository navigation
NVDA source code and user guide: remove Windows 8 constant mentions #15666
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 18 commits
e02bf58
9c6a627
f64a065
33ee059
8f3a519
f64a125
a040765
59e8c79
09a7957
23ddec0
4b61132
c43c9fc
7f6f111
458afd9
4dbbfbb
5c8a01e
2510085
b37b85a
96d4826
fad0b32
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,6 +16,8 @@ | |
| import functools | ||
| import winreg | ||
| import platform | ||
| import NVDAState | ||
| from logHandler import log | ||
|
|
||
|
|
||
| # Records a mapping between Windows builds and release names. | ||
|
|
@@ -211,10 +213,14 @@ def isUwpOcrAvailable(): | |
| return os.path.isdir(UWP_OCR_DATA_PATH) | ||
|
|
||
|
|
||
| def isFullScreenMagnificationAvailable() -> bool: | ||
| """ | ||
| Technically this is always False. The Magnification API has been marked by MS as unsupported for | ||
| WOW64 applications such as NVDA. For our usages, support has been added since Windows 8, relying on our | ||
| testing our specific usage of the API with each Windows version since Windows 8 | ||
| """ | ||
| return getWinVer() >= WIN8 | ||
| if NVDAState._allowDeprecatedAPI(): | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can this please use the standard
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I did it like this initially as the deprecated symbol is a function, not a constant or a variable, using gui.quit as precedence. If an add-on or other part of NVDA calls winVersion.isFullScreenMagnificationAvailable using the proposed deprecation method, Python will raise attribute error, which amounts to removing the function altogether from the module.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes sense
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this should give us a motivation to refine the getattr method when dealing with deprecated functions and class methods. I'll follow the suggested process, and then will create a new GitHub issue. |
||
| def isFullScreenMagnificationAvailable() -> bool: | ||
| """ | ||
| Technically this is always False. The Magnification API has been marked by MS as unsupported for | ||
| WOW64 applications such as NVDA. For our usages, support has been added since Windows 8, relying on our | ||
| testing our specific usage of the API with each Windows version since Windows 8 | ||
| """ | ||
| log.debugWarning( | ||
| "Deprecated function called: winVersion.isFullScreenMagnificationAvailable", stack_info=True | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this should suggest the alternative as well |
||
| ) | ||
| return True | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -117,6 +117,7 @@ Code which imports from one of them, should instead import from the replacement | |
|
|
||
| - Using ``watchdog.getFormattedStacksForAllThreads`` is deprecated - please use ``logHandler.getFormattedStacksForAllThreads`` instead. (#15616, @lukaszgo1) | ||
| - ``easeOfAccess.canConfigTerminateOnDesktopSwitch`` has been deprecated, as it became obsolete since Windows 7 is no longer supported. (#15644, @LeonarddeR) | ||
| - ``winVersion.isFullScreenMagnificationAvailable`` has been deprecated, as it became obsolete since Windows 7 is no longer supported. (#15664, @josephsl) | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this deprecation could also suggest |
||
| - | ||
|
|
||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.