Mouse Utils - Inclusive Mouse

[jaimecbernardo]

Summary of the Pull Request

What is this about:
Adds a new PowerToy called "Inclusive Mouse", under "Mouse Utilities".
This PowerToys draws a crosshair around the mouse pointer in its current screen, target at a set of users with specific accessibility needs.
This PowerToy has been designed originally by the accessibility team.
Default activation key is Ctrl+Alt+P.

What is included in the PR:

• The code for the PowerToys.
• Settings page and Oobe description line.
• Logging and telemetry.

How does someone test / validate:
Start PowerToys with the module enabled and press "Ctrl"+"Alt"+"P". Move the mouse around and check it has a crosshair following it "Ctrl"+"Alt"+"P" to toggle it off again.

Quality Checklist

• Linked issue: Mouse Utility - Crosshair mouse #15632
• Communication: I've discussed this with core contributors in the issue.
• Tests: Added/updated and all pass
• Installer: Added/updated and all pass
• Localization: All end user facing strings can be localized
• Docs: Added/ updated
• Binaries: Any new files are added to WXS / YML
  • No new binaries
  • YML for signing for new binaries
  • WXS for installer for new binaries

Contributor License Agreement (CLA)

A CLA must be signed. If not, go over here and sign the CLA.

[crutkas]

Trying to hand a build off to accessibility, possible to get fixes needed to get signing back on rebased on here?

[htcfreek]

@jaimecbernardo
On the oobe screenshot cursor isn't centered between the red lines. Is this a bug?

[jaimecbernardo]

@htcfreek , it's not a bug, it's as in the original internal design doc from the accessibility team.
It's centered around the point the mouse pointer is pointing to.

[crutkas]

Feedback: Off by default.

[crutkas]

One feedback was restricting width and border. If someone has it too high. It blocks everything. Thought was like 50 max

[jaimecbernardo]

One feedback was restricting width and border. If someone has it too high. It blocks everything. Thought was like 50 max

Added a maximum of 50 for the thickness and border size.

[htcfreek]

One feedback was restricting width and border. If someone has it too high. It blocks everything. Thought was like 50 max

Added a maximum of 50 for the thickness and border size.

Does minimum of 1 make sense? It will be invisible I think.

[jaimecbernardo]

One feedback was restricting width and border. If someone has it too high. It blocks everything. Thought was like 50 max

Added a maximum of 50 for the thickness and border size.

Does minimum of 1 make sense? It will be invisible I think.

Actually, I meant for the border size to have a minimum of 0.
Thickness is OK to have the minimum of 1. It's a straight 1 pixel-width line. Just tested it, it's still seen.

[stefansjfw]

what about upper limit for radius ? Doesn't make sense to set it to 10000 and make in invisible.

[jaimecbernardo]

what about upper limit for radius ? Doesn't make sense to set it to 10000 and make in invisible.

There are some big screens, but I guess a 500 limit wouldn't be outrageous.

[htcfreek]

@jaimecbernardo , @niels9001
Not sure if this is possible. But with the min/max value thing a realistic (respecting dpi and scaling) preview might be helpful.

[jaimecbernardo]

@jaimecbernardo , @niels9001 Not sure if this is possible. But with the min/max value thing a realistic (respecting dpi and scaling) preview might be helpful.

I think these is one of those things where we can let users pick weird settings if they want. What we don't want is for them to inadvertently block their whole screen and lose all visibility and not understand what happened ;)

[jaimecbernardo]

@htcfreek , Here's an idea of some big values being used. Max thickness/border in this case:

[crutkas]

I think we may want to just call this "Mouse pointer crosshairs" or something. Current name isn't really descriptive. We can adjust this before RTM but just calling it out.

[crutkas]

Pass name tweak, what else is needed

[jaimecbernardo]

Other than name tweak, everything is here.
Perhaps a different icon for settings, but it can be changed in a further PR.

[niels9001]

Other than name tweak, everything is here. Perhaps a different icon for settings, but it can be changed in a further PR.

Yeah, working on that.. I'll PR that in once this PR is merged.

[jaimecbernardo]

Other than name tweak, everything is here. Perhaps a different icon for settings, but it can be changed in a further PR.

Yeah, working on that.. I'll PR that in once this PR is merged.

Thanks a lot, Niels :)

[jaimecbernardo]

Just renamed the PowerToy to "Mouse Pointer Crosshair".
I'll merge this in as soon as CI passes.

[htcfreek]

Is Ctrl+Alt+Shift+P the default shortcut? Three modifier keys are a bit complicated to type.

[jaimecbernardo]

Is Ctrl+Alt+Shift+P the default shortcut? Three modifier keys are a bit complicated to type.

No, It's Ctrl+Alt+P. Those were my tests after the name change to make sure everything was working :)

[sntgchns]

Great addon! TNX!