add missing inputfield properties#83
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main-design-system #83 +/- ##
=====================================================
Coverage ? 28.48%
=====================================================
Files ? 139
Lines ? 4676
Branches ? 0
=====================================================
Hits ? 1332
Misses ? 3344
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@packages/stream_core_flutter/lib/src/components/message_composer/message_composer_input.dart`:
- Around line 79-83: The StreamMessageComposerInput currently hardcodes the
inner StreamMessageComposerInputField construction so callers cannot override
textInputAction, keyboardType, textCapitalization, autofocus, and autocorrect;
add these five properties to the StreamMessageComposerInput class/constructor
(matching the defaults used by StreamMessageComposerInputField, e.g.
TextCapitalization.sentences, autofocus=false, autocorrect=true) and forward
them when instantiating StreamMessageComposerInputField inside
StreamMessageComposerInput so the values are plumbed through to the inner field.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 02ed1925-95c1-4ccf-bacc-7e01cc623b50
📒 Files selected for processing (1)
packages/stream_core_flutter/lib/src/components/message_composer/message_composer_input.dart
Submit a pull request
CLA
Description of the pull request
This adds some missing fields in the inputfield that are available in the current chat sdk.
Summary by CodeRabbit