Merge pull request #227 from wordpress-mobile/issue/225-allow-images-in-block-elements

Issue/225 allow images in block elements

Author: 0nko

aztec/src/main/kotlin/org/wordpress/aztec/formatting/LineBlockFormatter.kt

@@ -270,14 +270,31 @@ class LineBlockFormatter(editor: AztecText, val headerStyle: LineBlockFormatter.
     }
 
     fun insertMedia(drawable: Drawable, source: String) {
-        val mediaStartIndex = selectionStart
-        val mediaEndIndex = selectionStart + source.length
 
-        editor.disableTextChangedListener()
+        val isAddingMediaToEndOfBlockElement = editableText.getSpans(selectionStart, selectionEnd, AztecBlockSpan::class.java)
+                .any {
+                    selectionStart == editableText.getSpanEnd(it)
+                }
+
+        val spanAtStartOfWhichMediaIsAdded = editableText.getSpans(selectionStart, selectionEnd, AztecBlockSpan::class.java)
+                .firstOrNull {
+                    selectionStart == editableText.getSpanStart(it)
+                }
+
+        val modifier = if (isAddingMediaToEndOfBlockElement) 1 else 0
+
+        val mediaStartIndex = selectionStart - modifier
+        val mediaEndIndex = selectionStart + source.length - modifier
+
+        if (!isAddingMediaToEndOfBlockElement)
+            editor.disableTextChangedListener()
+
         editableText.replace(selectionStart, selectionEnd, source)
 
-        editor.removeBlockStylesFromRange(mediaStartIndex, mediaEndIndex, true)
-        editor.removeHeadingStylesFromRange(mediaStartIndex, mediaEndIndex)
+        if (spanAtStartOfWhichMediaIsAdded != null) {
+            editableText.setSpan(spanAtStartOfWhichMediaIsAdded, mediaStartIndex, editableText.getSpanEnd(spanAtStartOfWhichMediaIsAdded), Spanned.SPAN_EXCLUSIVE_EXCLUSIVE)
+        }
+
         editor.removeInlineStylesFromRange(mediaStartIndex, mediaEndIndex)
 
         val span = AztecMediaSpan(editor.context, drawable, source)

aztec/src/main/kotlin/org/wordpress/aztec/spans/AztecListSpan.kt

@@ -51,7 +51,7 @@ abstract class AztecListSpan(val verticalPadding: Int) : LeadingMarginSpan.Stand
         val currentLineNumber = getIndexOfProcessedLine(text, end)
 
         if (totalNumberOfLinesInList > 1 && currentLineNumber == 1) {
-            adjustment = if (this is AztecOrderedListSpan) 0 else verticalPadding
+            adjustment = 0
         } else if ((totalNumberOfLinesInList > 1 && totalNumberOfLinesInList == currentLineNumber) || totalNumberOfLinesInList == 1) {
             adjustment = -verticalPadding
         }

aztec/src/main/kotlin/org/wordpress/aztec/spans/AztecUnorderedListSpan.kt

@@ -19,7 +19,6 @@ package org.wordpress.aztec.spans
 
 import android.graphics.Canvas
 import android.graphics.Paint
-import android.graphics.Path
 import android.text.Layout
 import android.text.Spanned
 import android.text.TextUtils
@@ -64,7 +63,6 @@ class AztecUnorderedListSpan : AztecListSpan {
     }
 
 
-
     override fun getLeadingMargin(first: Boolean): Int {
         return bulletMargin + 2 * bulletWidth + bulletPadding
     }
@@ -86,24 +84,14 @@ class AztecUnorderedListSpan : AztecListSpan {
         p.color = bulletColor
         p.style = Paint.Style.FILL
 
-        if (c.isHardwareAccelerated) {
-            bulletPath = Path()
-            bulletPath!!.addCircle(0.0f, 0.0f + getIndicatorAdjustment(text, end) / 2, bulletWidth.toFloat(), Path.Direction.CW)
+        val textToDraw = "\u2022"
 
-            c.save()
-            c.translate((x + bulletMargin + dir * bulletWidth).toFloat(), ((top + bottom) / 2.0f))
-            c.drawPath(bulletPath!!, p)
-            c.restore()
-        } else {
-            c.drawCircle((x + bulletMargin + dir * bulletWidth).toFloat(), ((top + bottom) / 2.0f) + getIndicatorAdjustment(text, end) / 2, bulletWidth.toFloat(), p)
-        }
+        val width = p.measureText(textToDraw)
+        c.drawText(textToDraw, (bulletMargin + x + dir - width) * dir, (bottom - p.descent()) - width / 2 + getIndicatorAdjustment(text, end), p)
 
         p.color = oldColor
         p.style = style
 
     }
 
-    companion object {
-        private var bulletPath: Path? = null
-    }
 }