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

Improve inputs #36

Closed
bdlukaa opened this issue May 20, 2021 · 1 comment · Fixed by #38
Closed

Improve inputs #36

bdlukaa opened this issue May 20, 2021 · 1 comment · Fixed by #38
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bdlukaa
Copy link
Owner

bdlukaa commented May 20, 2021

Currently, several input widgets uses a HoverButton as backend. Currently, it only supports one button state at the time, but it should support multiple. For example, when pressing and hovering, it should provide both to the builder callback.

@bdlukaa bdlukaa added the enhancement New feature or request label May 20, 2021
@bdlukaa bdlukaa self-assigned this May 20, 2021
@bdlukaa
Copy link
Owner Author

bdlukaa commented May 20, 2021

I think we should remove the HoverButton widget, and implement the mouse, gesture detector and focus directly on each input. This would add more boilerplate code, but would give us more control over these things in each widget. This is the approach used on material and cupertino libraries.

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

Successfully merging a pull request may close this issue.

1 participant