Fix crash when text is empty (#483)

Juanpe · Jan 18, 2022 · 9e61ff3 · 9e61ff3
1 parent 204b871
commit 9e61ff3
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/SkeletonViewCore/Sources/Internal/SkeletonExtensions/SkeletonTextNode.swift b/SkeletonViewCore/Sources/Internal/SkeletonExtensions/SkeletonTextNode.swift
@@ -106,9 +106,11 @@ extension UILabel: SkeletonTextNode {
     }
 
     var fontLineHeight: CGFloat? {
-        if let attributes = attributedText?.attributes(at: 0, effectiveRange: nil),
-           let fontAttribute = attributes.first(where: { $0.key == .font }) {
-            return fontAttribute.value as? CGFloat ?? font.lineHeight
+        if let attributedText = attributedText,
+           attributedText.length > 0 {
+            let attributes = attributedText.attributes(at: 0, effectiveRange: nil)
+            let fontAttribute = attributes.first(where: { $0.key == .font })
+            return fontAttribute?.value as? CGFloat ?? font.lineHeight
         } else {
             return font.lineHeight
         }
@@ -179,9 +181,11 @@ extension UITextView: SkeletonTextNode {
     }
 
     var fontLineHeight: CGFloat? {
-        if let attributes = attributedText?.attributes(at: 0, effectiveRange: nil),
-           let fontAttribute = attributes.first(where: { $0.key == .font }) {
-            return fontAttribute.value as? CGFloat ?? font?.lineHeight
+        if let attributedText = attributedText,
+           attributedText.length > 0 {
+            let attributes = attributedText.attributes(at: 0, effectiveRange: nil)
+            let fontAttribute = attributes.first(where: { $0.key == .font })
+            return fontAttribute?.value as? CGFloat ?? font?.lineHeight
         } else {
             return font?.lineHeight
         }