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

Use a pointer cursor when hovering the Show in File Manager button #37415

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Mar 29, 2020

This isn't consistent with other buttons, but helps people notice the button is actually clickable.

This partially addresses godotengine/godot-proposals#619.

This isn't consistent with other buttons, but helps people notice
the button is actually clickable.

This partially addresses
godotengine/godot-proposals#619.
@Calinou Calinou added this to the 4.0 milestone Mar 30, 2020
@Calinou Calinou added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Mar 30, 2020
@akien-mga
Copy link
Member

For me it's the same as #31620, I don't like hardcoding a custom cursor for only specific editor widgets, when there are hundreds of Buttons in the editor which won't follow the same logic.

Either this is made a feature of Button itself, or it's not used.
If Buttons don't appear clickable currently, that's a bug and it should be solved in the theme for all buttons, not just for this single one.

@Calinou
Copy link
Member Author

Calinou commented Feb 1, 2021

Superseded by #45642.

@Calinou Calinou closed this Feb 1, 2021
@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Feb 2, 2021
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