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

Add cursor property to the desktop super textfield. #384

Closed
wants to merge 1 commit into from

Conversation

salihgueler
Copy link
Contributor

Fixes #383

Right now the cursor for the default behavior of the SuperTextField is not configurable. For superlist, we would like to use custom cursors on hover to the field when there are no texts and there is no focus.

@matthew-carroll
Copy link
Contributor

Can you accomplish the same effect by wrapping a SuperTextField with a MouseRegion and configuring your cursor style in there?

@salihgueler
Copy link
Contributor Author

Can you accomplish the same effect by wrapping a SuperTextField with a MouseRegion and configuring your cursor style in there?

Nope, the cursor is absorbed by the SuperTextField.

@matthew-carroll
Copy link
Contributor

@salihgueler instead of taking in a configuration, please try setting the cursor to MouseCursor.defer when the user isn't over text, then try again with your own MouseRegion around the SuperTextField. Please let me know if that works.

Related docs: https://api.flutter.dev/flutter/widgets/MouseRegion/cursor.html

@matthew-carroll
Copy link
Contributor

@salihgueler I saw that @bk-one prioritized the ticket associated with this PR. Can you try the approach that I mentioned in my last comment and let me know if that solves the problem?

@matthew-carroll
Copy link
Contributor

@salihgueler can you try the "defer" approach running in a web browser and let me know if it works there? Based on some experimentation, I think this might be a Mac bug in Flutter.

@matthew-carroll
Copy link
Contributor

I'm going to close this PR due to inactivity. If someone wants to investigate this problem a little deeper, I welcome another attempt to solve this issue.

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

Successfully merging this pull request may close these issues.

Add cursor style support for SuperTextField
2 participants