Merge pull request #18 from Trendyol/bugfix/contentimagevisibility

Fix content image visibility bug
Trendyol · Jan 3, 2020 · 7a8784d · 7a8784d
2 parents 2a77307 + 16cf8dc
commit 7a8784d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/libraries/dialogs/src/main/java/com/trendyol/uicomponents/dialogs/Builder.kt b/libraries/dialogs/src/main/java/com/trendyol/uicomponents/dialogs/Builder.kt
@@ -15,7 +15,7 @@ open class InfoDialogBuilder internal constructor() : Builder() {
 
     var content: CharSequence = SpannableString("")
     @DrawableRes
-    var contentImage: Int = 0
+    var contentImage: Int? = null
     var showContentAsHtml: Boolean = false
 
     internal fun buildInfoDialog(block: InfoDialogBuilder.() -> Unit): DialogFragment {

diff --git a/libraries/dialogs/src/main/java/com/trendyol/uicomponents/dialogs/DialogViewState.kt b/libraries/dialogs/src/main/java/com/trendyol/uicomponents/dialogs/DialogViewState.kt
@@ -29,5 +29,5 @@ data class DialogViewState(
 
     fun isContentVisible(): Boolean = content.isNotEmpty()
 
-    fun isContentImageVisible(): Boolean = contentImage != null
+    fun isContentImageVisible(): Boolean = contentImage?.takeIf { it != 0 } != null
 }