-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Updated group policy docs and picture of File Explorer page #5051
Updated group policy docs and picture of File Explorer page #5051
Conversation
@acekirkpatrick : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit f1b957c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 99b32c0: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 3cadc35: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. We'll hold it until the release goes live. Thanks!
The new image cuts off the Thumbnail Preview part of the settings page. For a good overview about the module we should not do this. @acekirkpatrick can you update the new image? |
Learn Build status updates of commit 0de0c3b: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating the screenshot to include the entire screen!
@alvinashcraft |
Will do. On it now. |
@htcfreek Revert PR merged in private repo. It should sync to the public repo and live site in a few minutes. Thanks for the heads-up. |
…-group-policy-file-explorer-pr5051 Revert "Merge pull request #5051 from acekirkpatrick/FileExplorer"
Do we need a new PR from @acekirkpatrick to update the docs if it is implemented in PowerToys then? |
Yes, we'll need another PR to re-commit the changes in the next release. |
This is for the group policy changes in PowerToys #33475.
I also updated the screenshot for File Explorer to show the new toggle switch.
Please don't merge before: