Fixed link coloring

justjanne · Feb 4, 2019 · d46ab2f · d46ab2f
1 parent 0f1aadc
commit d46ab2f
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 7 deletions.
diff --git a/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/ContentFormatter.kt b/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/ContentFormatter.kt
@@ -127,7 +127,6 @@ class ContentFormatter @Inject constructor(
       }
     }
 
-    spans.reverse()
     for (span in spans) {
       span.apply(formattedText)
     }

diff --git a/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/model/IrcFormat.kt b/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/model/IrcFormat.kt
@@ -22,8 +22,6 @@ package de.kuschku.quasseldroid.util.irc.format.model
 import android.text.Spannable
 import android.text.Spanned
 import de.kuschku.libquassel.protocol.NetworkId
-import de.kuschku.quasseldroid.util.irc.format.ChannelLinkSpan
-import de.kuschku.quasseldroid.util.irc.format.QuasselURLSpan
 import de.kuschku.quasseldroid.util.irc.format.spans.*
 
 sealed class IrcFormat {

diff --git a/...ldroid/util/irc/format/ChannelLinkSpan.kt → .../util/irc/format/spans/ChannelLinkSpan.kt b/...ldroid/util/irc/format/ChannelLinkSpan.kt → .../util/irc/format/spans/ChannelLinkSpan.kt
@@ -17,7 +17,7 @@
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
-package de.kuschku.quasseldroid.util.irc.format
+package de.kuschku.quasseldroid.util.irc.format.spans
 
 import android.text.TextPaint
 import android.text.style.ClickableSpan

diff --git a/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/spans/IrcForegroundColorSpan.kt b/app/src/main/java/de/kuschku/quasseldroid/util/irc/format/spans/IrcForegroundColorSpan.kt
@@ -19,18 +19,26 @@
 
 package de.kuschku.quasseldroid.util.irc.format.spans
 
+import android.text.TextPaint
 import android.text.style.ForegroundColorSpan
 import androidx.annotation.ColorInt
 
 sealed class IrcForegroundColorSpan<T : IrcForegroundColorSpan<T>>(@ColorInt color: Int) :
   ForegroundColorSpan(color), Copyable<T> {
 
+  override fun updateDrawState(ds: TextPaint?) {
+    if (ds != null) {
+      ds.color = foregroundColor
+      ds.linkColor = foregroundColor
+    }
+  }
 
   class MIRC(private val mircColor: Int, @ColorInt color: Int) :
     IrcForegroundColorSpan<MIRC>(color), Copyable<MIRC> {
     override fun copy() = MIRC(mircColor, foregroundColor)
     override fun toString(): String {
-      return "IrcForegroundColorSpan.MIRC(mircColor=$mircColor, color=${foregroundColor.toString(16)})"
+      return "IrcForegroundColorSpan.MIRC(mircColor=$mircColor, color=${foregroundColor.toUInt().toString(
+        16)})"
     }
 
     override fun equals(other: Any?) = when (other) {
@@ -43,7 +51,7 @@ sealed class IrcForegroundColorSpan<T : IrcForegroundColorSpan<T>>(@ColorInt col
     IrcForegroundColorSpan<HEX>(color), Copyable<HEX> {
     override fun copy() = HEX(foregroundColor)
     override fun toString(): String {
-      return "IrcBackgroundColorSpan.HEX(color=${foregroundColor.toString(16)})"
+      return "IrcBackgroundColorSpan.HEX(color=${foregroundColor.toUInt().toString(16)})"
     }
 
     override fun equals(other: Any?) = when (other) {

diff --git a/...eldroid/util/irc/format/QuasselURLSpan.kt → ...d/util/irc/format/spans/QuasselURLSpan.kt b/...eldroid/util/irc/format/QuasselURLSpan.kt → ...d/util/irc/format/spans/QuasselURLSpan.kt
@@ -17,7 +17,7 @@
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
-package de.kuschku.quasseldroid.util.irc.format
+package de.kuschku.quasseldroid.util.irc.format.spans
 
 import android.text.TextPaint
 import android.text.style.URLSpan