Skip to content

fix: Remove null text when property is null#504

Merged
tushar-digia merged 1 commit intomainfrom
aditya/dg-318
Feb 12, 2026
Merged

fix: Remove null text when property is null#504
tushar-digia merged 1 commit intomainfrom
aditya/dg-318

Conversation

@aditya-digia
Copy link
Collaborator

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Walkthrough

A null-safety guard was added to handle null evaluated text by returning an empty SizedBox. A textValue variable was introduced to normalize string conversion using text.toString(), and this variable is now used throughout the widget construction instead of multiple text.toString() calls.

Changes

Cohort / File(s) Summary
Text widget null-safety and string normalization
lib/src/framework/widgets/text.dart
Added null-check guard for evaluated text, introduced textValue local variable to consolidate text.toString() calls, and optimized control flow by computing the value before widget construction.
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, so this cannot be evaluated for relevance or quality. Add a description explaining the null-safety fix, its impact, and any related issues or context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a null-safety guard for text properties to prevent null values from being rendered.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch aditya/dg-318

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tushar-digia tushar-digia merged commit 101f3fb into main Feb 12, 2026
1 of 2 checks passed
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.

3 participants