Skip to content
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

Closed
wants to merge 1 commit into from
Closed

Update screen-ruler.md default activation shortcut #4977

wants to merge 1 commit into from

Conversation

nx-frost
Copy link
Contributor

@nx-frost nx-frost commented May 5, 2024

Update default activation shortcut to ⊞ Win-Ctrl-Shift-M to resolve conflict with Windows Explorer built-in shortcut (microsoft/PowerToys#32623)

Copy link
Contributor

@nx-frost : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit f713ffe:

✅ Validation status: passed

File Status Preview URL Details
hub/powertoys/screen-ruler.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@alvinashcraft alvinashcraft left a 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.

@nx-frost nx-frost closed this by deleting the head repository Jun 16, 2024
@nx-frost
Copy link
Contributor Author

nx-frost commented Jul 4, 2024

@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.

@alvinashcraft
Copy link
Contributor

Hi @nx-frost. Let me see what I can do with this PR.

@alvinashcraft
Copy link
Contributor

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.

@nx-frost
Copy link
Contributor Author

I have opened #5064.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants