[Enterprise; Policy] Add policy for CursorWrap to ADMX#45028
Merged
vanzue merged 3 commits intomicrosoft:mainfrom Jan 27, 2026
Merged
[Enterprise; Policy] Add policy for CursorWrap to ADMX#45028vanzue merged 3 commits intomicrosoft:mainfrom
vanzue merged 3 commits intomicrosoft:mainfrom
Conversation
Collaborator
Author
|
@niels9001 |
niels9001
reviewed
Jan 24, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the missing Group Policy definition for the CursorWrap utility so it appears in ADMX/Intune administrative templates.
Changes:
- Bumped ADMX/ADML revision to
1.19and updated min required revision for resources. - Added
SUPPORTED_POWERTOYS_0_97_0supported-on entry. - Introduced the
ConfigureEnabledUtilityCursorWrappolicy definition and its localized display string.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/gpo/assets/en-US/PowerToys.adml |
Updates revision and adds localized strings for v0.97.0 support and the CursorWrap enabled-state policy display name. |
src/gpo/assets/PowerToys.admx |
Updates revision/minRequiredRevision, adds v0.97.0 supported-on definition, and adds the CursorWrap enabled-state policy entry. |
| <string id="ConfigureEnabledUtilityKeyboardManager">Keyboard Manager: Configure enabled state</string> | ||
| <string id="ConfigureEnabledUtilityFindMyMouse">Find My Mouse: Configure enabled state</string> | ||
| <string id="ConfigureEnabledUtilityMouseHighlighter">Mouse Highlighter: Configure enabled state</string> | ||
| <string id="ConfigureEnabledUtilityCursorWrap">CursorWrap: Configure enabled state</string> |
There was a problem hiding this comment.
The display string for this policy uses "CursorWrap" without a space. Elsewhere in the codebase the feature is referred to as "Cursor Wrap" (e.g., cmdpal Toggle Cursor Wrap strings), and using the spaced name makes the policy more readable and easier to find via search. Consider changing this to "Cursor Wrap: Configure enabled state" while keeping the string id/policy name as-is.
Suggested change
| <string id="ConfigureEnabledUtilityCursorWrap">CursorWrap: Configure enabled state</string> | |
| <string id="ConfigureEnabledUtilityCursorWrap">Cursor Wrap: Configure enabled state</string> |
Collaborator
Author
There was a problem hiding this comment.
@niels9001
I thought I have seen the spaced version somewhere. But I can't remember where. What is the correct name?
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
vanzue
approved these changes
Jan 27, 2026
| <definition name="SUPPORTED_POWERTOYS_0_89_0" displayName="$(string.SUPPORTED_POWERTOYS_0_89_0)"/> | ||
| <definition name="SUPPORTED_POWERTOYS_0_90_0" displayName="$(string.SUPPORTED_POWERTOYS_0_90_0)"/> | ||
| <definition name="SUPPORTED_POWERTOYS_0_96_0" displayName="$(string.SUPPORTED_POWERTOYS_0_96_0)"/> | ||
| <definition name="SUPPORTED_POWERTOYS_0_97_0" displayName="$(string.SUPPORTED_POWERTOYS_0_97_0)"/> |
Contributor
There was a problem hiding this comment.
Shall we mark the version as 0_97_1?
Contributor
There was a problem hiding this comment.
I realize our code has already support the cursorwrap, so we are good to use 0.97 here?
vanzue
pushed a commit
that referenced
this pull request
Jan 27, 2026
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Added missing policy definition. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist - [x] Closes: #44897 <!-- - [ ] Closes: #yyy (add separate lines for additional resolved issues) --> - [x] **Communication:** I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected - [ ] **Tests:** Added/updated and all pass - [x] **Localization:** All end-user-facing strings can be localized - [ ] **Dev docs:** Added/updated - [ ] **New binaries:** Added on the required places - [ ] [JSON for signing](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ESRPSigning_core.json) for new binaries - [ ] [WXS for installer](https://github.com/microsoft/PowerToys/blob/main/installer/PowerToysSetup/Product.wxs) for new binaries and localization folder - [ ] [YML for CI pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ci/templates/build-powertoys-steps.yml) for new test projects - [ ] [YML for signed pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/release.yml) - [x] **Documentation updated:** See PR for issue #44484 <!-- Provide a more detailed description of the PR, other things fixed, or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed
yeelam-gordon
pushed a commit
that referenced
this pull request
Jan 31, 2026
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Added missing policy definition. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist - [x] Closes: #44897 <!-- - [ ] Closes: #yyy (add separate lines for additional resolved issues) --> - [x] **Communication:** I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected - [ ] **Tests:** Added/updated and all pass - [x] **Localization:** All end-user-facing strings can be localized - [ ] **Dev docs:** Added/updated - [ ] **New binaries:** Added on the required places - [ ] [JSON for signing](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ESRPSigning_core.json) for new binaries - [ ] [WXS for installer](https://github.com/microsoft/PowerToys/blob/main/installer/PowerToysSetup/Product.wxs) for new binaries and localization folder - [ ] [YML for CI pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ci/templates/build-powertoys-steps.yml) for new test projects - [ ] [YML for signed pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/release.yml) - [x] **Documentation updated:** See PR for issue #44484 <!-- Provide a more detailed description of the PR, other things fixed, or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of the Pull Request
Added missing policy definition.
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed