-
-
Notifications
You must be signed in to change notification settings - Fork 77
feat: add support for inkwell widget #241
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
feat: add support for inkwell widget #241
Conversation
Created StacInkWell and StacInkwellParser for Inkwell widget support
…ties Added support for onDoubleTap, onLongPress, onTapDown, onTapUp, onTapCancel, onSecondaryTap, onSecondaryTapUp, onSecondaryTapDown, onSecondaryTapCancel, onHighlightChanged, and onHover in the StacInkWell model.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @AbhijithKonnayil 💯
Please check my suggestions.
packages/stac/lib/src/parsers/widgets/stac_inkwell/stac_inkwell_parser.dart
Show resolved
Hide resolved
packages/stac/lib/src/parsers/widgets/stac_inkwell/stac_inkwell_parser.dart
Show resolved
Hide resolved
updated the PR with your suggestions. Please cheak |
Hey @AbhijithKonnayil!! Looks like PR format check is failing. Can you please check? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution @AbhijithKonnayil 🥇
Description
Add support for Inkwell
Related Issues #238
Type of Change