-
Notifications
You must be signed in to change notification settings - Fork 1.4k
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
[Feature] Add GazeInput.SetCursorBrush method to change the default GazeCursor color #3210
Comments
Thanks for submitting a new feature request! I've automatically added a vote 👍 reaction to help get things started. Other community members can vote to help us prioritize this feature in the future! |
@niels9001 looks like you copied the signature, did you mean it to be? public static void SetCursorBrush(UIElement, Brush value); I don't know much about the internals of the Gaze helpers, @peteams thoughts? Is this easy to do? |
Oh yeah, sorry about that.. fixed! I have true lack of C++ knowledge, else I would have put in a PR myself :)! I see in the code that an Ellipse is drawn and the fill is set to IndianRed. Not sure if we could create a simple method that overrides that (like the SetCursorRadius already does)? |
Id really like an opportunity to work on this one if thats OK with everyone else |
That would be very much appreciated :) |
Does this need to be assigned to me? Or shall I just start work on it :) |
@deanchalk I just assigned it to you. Thanks for the help! |
…he default GazeCursor color
OK, I have had a chance to look at this. The C++ control code is fairly straightforward, but we can achieve the desired result using the current GazeInteraction APIs. HOWEVER.... I do not have a gaze tracking device, so cannot actually test this. The code change seems like it should do the job, when you delve into the C++ it seems very clear that this is what you have to do, but who knows - I may have coded something wrong. |
Thanks for the clarification! What's the PR no.? I can try it out to see if it works. |
To be honest, the c++ code could do with some improvements regarding it's API surface, plus ideally it would be upgraded from C++/CX to C++/WinRT. If I ever invest in an eye tracker I'd be happy to re-work this set of controls. Hopefully the PR I have submitted offers a working solution :) |
Haven't heard from @deanchalk lately, hope everything is alright. Later last year for 7.0.0 we ported the gaze code to C#, so it should be easier to modify directly in the future for improvements like this one. In the meantime, if anyone wants to take on taking a look at adding this function, let us know. |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Describe the problem this feature would solve
The current Toolkit implementation allows for setting the visibility of the gaze cursor and the radius of the ellipse.
The brush however, can not be changed - which is unfortunate because the default color does look great and having the flexibility to change it (for e.g. dark vs light themed UIs) to create more accessible apps would be great!
Describe the solution
The current available methods are:
public static void SetIsCursorVisible(UIElement element, bool value);
public static void SetCursorRadius(UIElement element, int value);
The proposed method would be:
public static void SetCursorBrush(UIElement element, Brush value);
Additional context & Screenshots
The text was updated successfully, but these errors were encountered: