-
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
Update screen-ruler.md default activation shortcut #4977
Conversation
@nx-frost : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit f713ffe: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
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, This update looks good. I will hold this PR until the shortcut update is released in PowerToys.
@alvinashcraft I deleted my fork of this repo after the change was merged into PowerToys 0.82, however I am noticing this documentation was not merged. Do I need to re-create the PR, or can it be pulled as-is? Thanks. |
Hi @nx-frost. Let me see what I can do with this PR. |
I'm not seeing a simple way to use the existing PR or commit to merge this change. I could create a PR myself, but I'd rather have you get credit for the change in the git history. Do you want to submit another PR for this, @nx-frost. |
I have opened #5064. |
Update default activation shortcut to
⊞ Win
-Ctrl
-Shift
-M
to resolve conflict with Windows Explorer built-in shortcut (microsoft/PowerToys#32623)