fix(android): use explicit null/undefined check and update type in colorProperty.setNative

Copilot · NathanWalker · web-flow · commit 9bc2fbf5397d · 2026-03-27T22:14:53.000Z
Agent-Logs-Url: https://github.com/NativeScript/NativeScript/sessions/bb1580df-8f26-4e42-85b4-2c73c1bc54bc Co-authored-by: NathanWalker <457187+NathanWalker@users.noreply.github.com>
diff --git a/packages/core/ui/text-base/index.android.ts b/packages/core/ui/text-base/index.android.ts
@@ -408,11 +408,11 @@ export class TextBase extends TextBaseCommon {
 	[colorProperty.getDefault](): android.content.res.ColorStateList {
 		return this.nativeTextViewProtected.getTextColors();
 	}
-	[colorProperty.setNative](value: Color | android.content.res.ColorStateList) {
+	[colorProperty.setNative](value: Color | android.content.res.ColorStateList | null | undefined) {
 		if (!this.formattedText || !(value instanceof Color)) {
 			if (value instanceof Color) {
 				this.nativeTextViewProtected.setTextColor(value.android);
-			} else if (value) {
+			} else if (value != null) {
 				this.nativeTextViewProtected.setTextColor(value);
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -408,11 +408,11 @@ export class TextBase extends TextBaseCommon {`
`408`	`408`	`[colorProperty.getDefault](): android.content.res.ColorStateList {`
`409`	`409`	`return this.nativeTextViewProtected.getTextColors();`
`410`	`410`	`}`
`411`		`- [colorProperty.setNative](value: Color \| android.content.res.ColorStateList) {`
	`411`	`+ [colorProperty.setNative](value: Color \| android.content.res.ColorStateList \| null \| undefined) {`
`412`	`412`	`if (!this.formattedText \|\| !(value instanceof Color)) {`
`413`	`413`	`if (value instanceof Color) {`
`414`	`414`	`this.nativeTextViewProtected.setTextColor(value.android);`
`415`		`- } else if (value) {`
	`415`	`+ } else if (value != null) {`
`416`	`416`	`this.nativeTextViewProtected.setTextColor(value);`
`417`	`417`	`}`
`418`	`418`	`}`