-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add Visuals::interact_cursor
#3312
Conversation
This should be opt-in, set as an option in I personally prefer pointing-hand to mean "clicking here will bring you someplace else", which not all buttons do, but of course this is a matter of preference. You should also consider if this should apply to other widgets too. Maybe you want anything that is clickable to have the pointing hand? |
Makes sense to me. I'll look into making this a visual option and take a look at other clickable widgets in egui. Should be committed by the end of the day. Thanks for such a fast review! |
|
Sounds good to me! I'm putting this into an option now. |
This is now an option that is off by default. From what I can tell browsers don't invoke this behaviour when hovering over a checkbox or a radio button. It's just for normal buttons. There's an argument to be made that things like
|
Nope - it's only for hyperlinks. Sites override this for buttons, but for desktop UI elements this isn't common.
|
Oh wow, yea. Everything I checked this on overrides that functionality. I guess I've just subconsciously gotten used to it and thought it was the default.
Yeah this is a huge improvement. One sec. |
Visuals::interact_cursor
When hovering over an
egui::Button
, the cursor should change to a PointingHand. This is consistent with the buttons in most other UIs (the browser, etc.)Closes #3311.