-
Notifications
You must be signed in to change notification settings - Fork 6
feat(UX-1501): message input #384
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
base: main
Are you sure you want to change the base?
Conversation
PR Checks complete
Created with Flutter code quality action |
…ality improvements - Refactored action buttons to include disabled state and semantic labels. - Implemented disabled state for ZetaMessageInput, affecting all action buttons and input fields. - Improved test coverage for message input component, including accessibility and interaction tests. - Updated widgetbook examples to reflect new properties and functionality.
|
Visit the preview URL for this PR (updated for commit 92ba34e): https://zeta-flutter-main--pr-384-ux-1501-voxans6c.web.app (expires Fri, 26 Sep 2025 14:48:55 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 5ca681de0a0ad9185b252304c113355d5ee04ca6 |
thelukewalton
left a comment
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.
Need to test on iOS - seems like location, camera, voice memo do not work
| return ActionButton( | ||
| icon: ZetaIcons.audio, | ||
| onPressed: onSend != null ? () => _onClick(context) : null, | ||
| semanticLabel: 'record a voice memo', |
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.
this should be a parameter passed into the widget
Actions include:
Photo attachmentVideo attachment