From 4458e28ce27c142a28ac87a5dd2fed521bb60131 Mon Sep 17 00:00:00 2001
From: ganfra <francois.ganard@gmail.com>
Date: Thu, 21 Feb 2019 19:21:08 +0100
Subject: [PATCH 01/13] Html : introduce markown lib to handle html rendering.

---
 app/build.gradle                              |  4 ++
 .../vector/riotredesign/core/di/AppModule.kt  |  5 ++
 .../riotredesign/features/home/HomeModule.kt  |  2 +-
 .../detail/timeline/MessageItemFactory.kt     | 14 ++++-
 .../features/markdown/HtmlRenderer.kt         | 63 +++++++++++++++++++
 build.gradle                                  |  1 +
 6 files changed, 85 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt

diff --git a/app/build.gradle b/app/build.gradle
index 12b174e0b5..e91f74d450 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -60,6 +60,7 @@ dependencies {
 
     def epoxy_version = "3.0.0"
     def arrow_version = "0.8.2"
+    def markwon_version = '3.0.0-SNAPSHOT'
 
     implementation project(":matrix-sdk-android")
     implementation project(":matrix-sdk-android-rx")
@@ -97,6 +98,9 @@ dependencies {
     implementation 'com.amulyakhare:com.amulyakhare.textdrawable:1.0.1'
     implementation 'com.google.android.material:material:1.1.0-alpha02'
     implementation 'me.gujun.android:span:1.7'
+    implementation "ru.noties.markwon:core:$markwon_version"
+    implementation "ru.noties.markwon:html:$markwon_version"
+
 
     // DI
     implementation "org.koin:koin-android:$koin_version"
diff --git a/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt b/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
index bbc973152c..b022525188 100644
--- a/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
+++ b/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
@@ -22,6 +22,7 @@ import im.vector.riotredesign.core.resources.ColorProvider
 import im.vector.riotredesign.core.resources.LocaleProvider
 import im.vector.riotredesign.core.resources.StringProvider
 import im.vector.riotredesign.features.home.room.list.RoomSelectionRepository
+import im.vector.riotredesign.features.markdown.HtmlRenderer
 import org.koin.dsl.module.module
 
 class AppModule(private val context: Context) {
@@ -48,5 +49,9 @@ class AppModule(private val context: Context) {
             RoomSelectionRepository(get())
         }
 
+        single {
+            HtmlRenderer(context)
+        }
+
     }
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
index 2f2d85a026..1af5f66467 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
@@ -33,7 +33,7 @@ class HomeModule {
         }
 
         single {
-            MessageItemFactory(get(), get(), get())
+            MessageItemFactory(get(), get(), get(), get())
         }
 
         single {
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
index e2f61cf0c8..2aae81d899 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
@@ -29,12 +29,14 @@ import im.vector.riotredesign.core.epoxy.RiotEpoxyModel
 import im.vector.riotredesign.core.extensions.localDateTime
 import im.vector.riotredesign.core.resources.ColorProvider
 import im.vector.riotredesign.features.home.room.detail.timeline.helper.TimelineMediaSizeProvider
+import im.vector.riotredesign.features.markdown.HtmlRenderer
 import im.vector.riotredesign.features.media.MediaContentRenderer
 import me.gujun.android.span.span
 
 class MessageItemFactory(private val colorProvider: ColorProvider,
                          private val timelineMediaSizeProvider: TimelineMediaSizeProvider,
-                         private val timelineDateFormatter: TimelineDateFormatter) {
+                         private val timelineDateFormatter: TimelineDateFormatter,
+                         private val htmlRenderer: HtmlRenderer) {
 
     private val messagesDisplayedWithInformation = HashSet<String?>()
 
@@ -102,9 +104,15 @@ class MessageItemFactory(private val colorProvider: ColorProvider,
                                      informationData: MessageInformationData,
                                      callback: TimelineEventController.Callback?): MessageTextItem? {
 
-        val message = linkifyBody(messageContent.body, callback)
+        val bodyToUse = messageContent.formattedBody
+                ?.let {
+                    htmlRenderer.render(it)
+                }
+                ?: messageContent.body
+
+        val linkifiedBody = linkifyBody(bodyToUse, callback)
         return MessageTextItem_()
-                .message(message)
+                .message(linkifiedBody)
                 .informationData(informationData)
     }
 
diff --git a/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
new file mode 100644
index 0000000000..f3c659333a
--- /dev/null
+++ b/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
@@ -0,0 +1,63 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.riotredesign.features.markdown
+
+import android.content.Context
+import ru.noties.markwon.AbstractMarkwonPlugin
+import ru.noties.markwon.Markwon
+import ru.noties.markwon.MarkwonVisitor
+import ru.noties.markwon.html.HtmlPlugin
+import ru.noties.markwon.html.HtmlTag
+import ru.noties.markwon.html.MarkwonHtmlRenderer
+import ru.noties.markwon.html.TagHandler
+import timber.log.Timber
+
+class HtmlRenderer(private val context: Context) {
+
+    private val markwon = Markwon.builder(context)
+            .usePlugin(HtmlPlugin.create())
+            .usePlugin(MatrixPlugin.create())
+            .build()
+
+    fun render(text: String): CharSequence {
+        return markwon.toMarkdown(text)
+    }
+
+}
+
+private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
+
+    override fun configureHtmlRenderer(builder: MarkwonHtmlRenderer.Builder) {
+        builder.addHandler("mx-reply", MxReplyTagHandler())
+    }
+
+    companion object {
+
+        fun create(): MatrixPlugin {
+            return MatrixPlugin()
+        }
+    }
+}
+
+private class MxReplyTagHandler : TagHandler() {
+    override fun handle(visitor: MarkwonVisitor, renderer: MarkwonHtmlRenderer, tag: HtmlTag) {
+        Timber.v("Handle mx-reply")
+    }
+
+}
\ No newline at end of file
diff --git a/build.gradle b/build.gradle
index 7216b2637d..af25b46084 100644
--- a/build.gradle
+++ b/build.gradle
@@ -22,6 +22,7 @@ allprojects {
         google()
         jcenter()
         maven { url 'https://jitpack.io' }
+        maven { url 'https://oss.sonatype.org/content/repositories/snapshots/' }
     }
 }
 

From d2db5e32fcace618556e79d6ac31b81d9c8487bd Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Fri, 22 Feb 2019 15:43:48 +0100
Subject: [PATCH 02/13] Html : start handling reply

---
 .../features/markdown/HtmlRenderer.kt         | 97 +++++++++++++++++--
 1 file changed, 91 insertions(+), 6 deletions(-)

diff --git a/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
index f3c659333a..30f5030ea9 100644
--- a/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
@@ -19,19 +19,35 @@
 package im.vector.riotredesign.features.markdown
 
 import android.content.Context
+import org.commonmark.node.BlockQuote
+import org.commonmark.node.HtmlBlock
+import org.commonmark.node.HtmlInline
+import org.commonmark.node.Node
 import ru.noties.markwon.AbstractMarkwonPlugin
 import ru.noties.markwon.Markwon
+import ru.noties.markwon.MarkwonConfiguration
 import ru.noties.markwon.MarkwonVisitor
-import ru.noties.markwon.html.HtmlPlugin
+import ru.noties.markwon.SpannableBuilder
 import ru.noties.markwon.html.HtmlTag
+import ru.noties.markwon.html.MarkwonHtmlParserImpl
 import ru.noties.markwon.html.MarkwonHtmlRenderer
 import ru.noties.markwon.html.TagHandler
-import timber.log.Timber
+import ru.noties.markwon.html.tag.BlockquoteHandler
+import ru.noties.markwon.html.tag.EmphasisHandler
+import ru.noties.markwon.html.tag.HeadingHandler
+import ru.noties.markwon.html.tag.ImageHandler
+import ru.noties.markwon.html.tag.LinkHandler
+import ru.noties.markwon.html.tag.ListHandler
+import ru.noties.markwon.html.tag.StrikeHandler
+import ru.noties.markwon.html.tag.StrongEmphasisHandler
+import ru.noties.markwon.html.tag.SubScriptHandler
+import ru.noties.markwon.html.tag.SuperScriptHandler
+import ru.noties.markwon.html.tag.UnderlineHandler
+import java.util.Arrays.asList
 
 class HtmlRenderer(private val context: Context) {
 
     private val markwon = Markwon.builder(context)
-            .usePlugin(HtmlPlugin.create())
             .usePlugin(MatrixPlugin.create())
             .build()
 
@@ -43,8 +59,65 @@ class HtmlRenderer(private val context: Context) {
 
 private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
 
+    override fun configureConfiguration(builder: MarkwonConfiguration.Builder) {
+        builder.htmlParser(MarkwonHtmlParserImpl.create())
+    }
+
     override fun configureHtmlRenderer(builder: MarkwonHtmlRenderer.Builder) {
-        builder.addHandler("mx-reply", MxReplyTagHandler())
+        builder
+                .addHandler(
+                        "img",
+                        ImageHandler.create())
+                .addHandler(
+                        "a",
+                        LinkHandler())
+                .addHandler(
+                        "blockquote",
+                        BlockquoteHandler())
+                .addHandler(
+                        "sub",
+                        SubScriptHandler())
+                .addHandler(
+                        "sup",
+                        SuperScriptHandler())
+                .addHandler(
+                        asList<String>("b", "strong"),
+                        StrongEmphasisHandler())
+                .addHandler(
+                        asList<String>("s", "del"),
+                        StrikeHandler())
+                .addHandler(
+                        asList<String>("u", "ins"),
+                        UnderlineHandler())
+                .addHandler(
+                        asList<String>("ul", "ol"),
+                        ListHandler())
+                .addHandler(
+                        asList<String>("i", "em", "cite", "dfn"),
+                        EmphasisHandler())
+                .addHandler(
+                        asList<String>("h1", "h2", "h3", "h4", "h5", "h6"),
+                        HeadingHandler())
+                .addHandler("mx-reply",
+                            MxReplyTagHandler())
+
+    }
+
+    override fun afterRender(node: Node, visitor: MarkwonVisitor) {
+        val configuration = visitor.configuration()
+        configuration.htmlRenderer().render(visitor, configuration.htmlParser())
+    }
+
+    override fun configureVisitor(builder: MarkwonVisitor.Builder) {
+        builder
+                .on(HtmlBlock::class.java) { visitor, htmlBlock -> visitHtml(visitor, htmlBlock.literal) }
+                .on(HtmlInline::class.java) { visitor, htmlInline -> visitHtml(visitor, htmlInline.literal) }
+    }
+
+    private fun visitHtml(visitor: MarkwonVisitor, html: String?) {
+        if (html != null) {
+            visitor.configuration().htmlParser().processFragment(visitor.builder(), html)
+        }
     }
 
     companion object {
@@ -56,8 +129,20 @@ private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
 }
 
 private class MxReplyTagHandler : TagHandler() {
+
     override fun handle(visitor: MarkwonVisitor, renderer: MarkwonHtmlRenderer, tag: HtmlTag) {
-        Timber.v("Handle mx-reply")
+        val configuration = visitor.configuration()
+        val factory = configuration.spansFactory().get(BlockQuote::class.java)
+        if (factory != null) {
+            SpannableBuilder.setSpans(
+                    visitor.builder(),
+                    factory.getSpans(configuration, visitor.renderProps()),
+                    tag.start(),
+                    tag.end()
+            )
+            val replyText = visitor.builder().removeFromEnd(tag.end())
+            visitor.builder().append("\n\n").append(replyText)
+        }
     }
 
-}
\ No newline at end of file
+}

From f06211ce4fb3059e44e65dac76d52495ecd2c7b8 Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Mon, 25 Feb 2019 15:18:36 +0100
Subject: [PATCH 03/13] Start playing with ChipDrawable to show Pills

---
 .../vector/riotredesign/core/di/AppModule.kt  |  5 --
 .../features/home/HomeActivity.kt             |  2 +-
 .../riotredesign/features/home/HomeModule.kt  |  7 ++-
 .../detail/timeline/MessageItemFactory.kt     |  4 +-
 .../{HtmlRenderer.kt => EventHtmlRenderer.kt} | 49 ++++++++++++++++---
 app/src/main/res/values/colors.xml            |  5 ++
 app/src/main/res/values/themes_base.xml       |  3 +-
 app/src/main/res/xml/pill_view.xml            |  6 +++
 8 files changed, 64 insertions(+), 17 deletions(-)
 rename app/src/main/java/im/vector/riotredesign/features/markdown/{HtmlRenderer.kt => EventHtmlRenderer.kt} (72%)
 create mode 100644 app/src/main/res/xml/pill_view.xml

diff --git a/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt b/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
index b022525188..bbc973152c 100644
--- a/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
+++ b/app/src/main/java/im/vector/riotredesign/core/di/AppModule.kt
@@ -22,7 +22,6 @@ import im.vector.riotredesign.core.resources.ColorProvider
 import im.vector.riotredesign.core.resources.LocaleProvider
 import im.vector.riotredesign.core.resources.StringProvider
 import im.vector.riotredesign.features.home.room.list.RoomSelectionRepository
-import im.vector.riotredesign.features.markdown.HtmlRenderer
 import org.koin.dsl.module.module
 
 class AppModule(private val context: Context) {
@@ -49,9 +48,5 @@ class AppModule(private val context: Context) {
             RoomSelectionRepository(get())
         }
 
-        single {
-            HtmlRenderer(context)
-        }
-
     }
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
index 087a00eaec..7f1e00d3ad 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
@@ -45,7 +45,7 @@ class HomeActivity : RiotActivity(), ToolbarConfigurable {
     private val homeNavigator by inject<HomeNavigator>()
 
     override fun onCreate(savedInstanceState: Bundle?) {
-        loadKoinModules(listOf(HomeModule().definition))
+        loadKoinModules(listOf(HomeModule(this).definition))
         homeNavigator.activity = this
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_home)
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
index 1af5f66467..35c8276cd8 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
@@ -22,9 +22,10 @@ import im.vector.riotredesign.features.home.room.detail.timeline.*
 import im.vector.riotredesign.features.home.room.detail.timeline.helper.TimelineMediaSizeProvider
 import im.vector.riotredesign.features.home.room.list.RoomSummaryComparator
 import im.vector.riotredesign.features.home.room.list.RoomSummaryController
+import im.vector.riotredesign.features.markdown.EventHtmlRenderer
 import org.koin.dsl.module.module
 
-class HomeModule {
+class HomeModule(homeActivity: HomeActivity) {
 
     val definition = module(override = true) {
 
@@ -32,6 +33,10 @@ class HomeModule {
             TimelineDateFormatter(get())
         }
 
+        single {
+            EventHtmlRenderer(homeActivity)
+        }
+
         single {
             MessageItemFactory(get(), get(), get(), get())
         }
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
index 2aae81d899..e1eaaecc6f 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
@@ -29,14 +29,14 @@ import im.vector.riotredesign.core.epoxy.RiotEpoxyModel
 import im.vector.riotredesign.core.extensions.localDateTime
 import im.vector.riotredesign.core.resources.ColorProvider
 import im.vector.riotredesign.features.home.room.detail.timeline.helper.TimelineMediaSizeProvider
-import im.vector.riotredesign.features.markdown.HtmlRenderer
+import im.vector.riotredesign.features.markdown.EventHtmlRenderer
 import im.vector.riotredesign.features.media.MediaContentRenderer
 import me.gujun.android.span.span
 
 class MessageItemFactory(private val colorProvider: ColorProvider,
                          private val timelineMediaSizeProvider: TimelineMediaSizeProvider,
                          private val timelineDateFormatter: TimelineDateFormatter,
-                         private val htmlRenderer: HtmlRenderer) {
+                         private val htmlRenderer: EventHtmlRenderer) {
 
     private val messagesDisplayedWithInformation = HashSet<String?>()
 
diff --git a/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt
similarity index 72%
rename from app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
rename to app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt
index 30f5030ea9..206d4282be 100644
--- a/app/src/main/java/im/vector/riotredesign/features/markdown/HtmlRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt
@@ -19,6 +19,11 @@
 package im.vector.riotredesign.features.markdown
 
 import android.content.Context
+import android.text.style.ImageSpan
+import com.google.android.material.chip.ChipDrawable
+import im.vector.matrix.android.api.permalinks.PermalinkData
+import im.vector.matrix.android.api.permalinks.PermalinkParser
+import im.vector.riotredesign.R
 import org.commonmark.node.BlockQuote
 import org.commonmark.node.HtmlBlock
 import org.commonmark.node.HtmlInline
@@ -45,10 +50,10 @@ import ru.noties.markwon.html.tag.SuperScriptHandler
 import ru.noties.markwon.html.tag.UnderlineHandler
 import java.util.Arrays.asList
 
-class HtmlRenderer(private val context: Context) {
+class EventHtmlRenderer(private val context: Context) {
 
     private val markwon = Markwon.builder(context)
-            .usePlugin(MatrixPlugin.create())
+            .usePlugin(MatrixPlugin.create(context))
             .build()
 
     fun render(text: String): CharSequence {
@@ -57,7 +62,7 @@ class HtmlRenderer(private val context: Context) {
 
 }
 
-private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
+private class MatrixPlugin private constructor(private val context: Context) : AbstractMarkwonPlugin() {
 
     override fun configureConfiguration(builder: MarkwonConfiguration.Builder) {
         builder.htmlParser(MarkwonHtmlParserImpl.create())
@@ -70,7 +75,7 @@ private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
                         ImageHandler.create())
                 .addHandler(
                         "a",
-                        LinkHandler())
+                        MxLinkHandler(context))
                 .addHandler(
                         "blockquote",
                         BlockquoteHandler())
@@ -122,12 +127,44 @@ private class MatrixPlugin private constructor() : AbstractMarkwonPlugin() {
 
     companion object {
 
-        fun create(): MatrixPlugin {
-            return MatrixPlugin()
+        fun create(context: Context): MatrixPlugin {
+            return MatrixPlugin(context)
         }
     }
 }
 
+private class MxLinkHandler(private val context: Context) : TagHandler() {
+
+    private val linkHandler = LinkHandler()
+
+    override fun handle(visitor: MarkwonVisitor, renderer: MarkwonHtmlRenderer, tag: HtmlTag) {
+        val link = tag.attributes()["href"]
+        if (link != null) {
+            val permalinkData = PermalinkParser.parse(link)
+            when (permalinkData) {
+                is PermalinkData.UserLink -> {
+                    val chipDrawable = ChipDrawable.createFromResource(context, R.xml.pill_view)
+                    chipDrawable.setText(permalinkData.userId)
+                    chipDrawable.textEndPadding = 8f
+                    chipDrawable.textStartPadding = 8f
+                    chipDrawable.setBounds(0, 0, chipDrawable.intrinsicWidth, (chipDrawable.intrinsicHeight / 1.5f).toInt())
+                    val span = ImageSpan(chipDrawable)
+                    SpannableBuilder.setSpans(
+                            visitor.builder(),
+                            span,
+                            tag.start(),
+                            tag.end()
+                    )
+                }
+                else                      -> linkHandler.handle(visitor, renderer, tag)
+            }
+        } else {
+            linkHandler.handle(visitor, renderer, tag)
+        }
+    }
+
+}
+
 private class MxReplyTagHandler : TagHandler() {
 
     override fun handle(visitor: MarkwonVisitor, renderer: MarkwonHtmlRenderer, tag: HtmlTag) {
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index 9797d9459d..90e3e22a0b 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -17,4 +17,9 @@
     <color name="pale_grey_two">#ebedf8</color>
     <color name="brown_grey">#a5a5a5</color>
     <color name="grey_lynch">#61708B</color>
+
+    <color name="vector_silver_color">#FFC7C7C7</color>
+    <color name="vector_dark_grey_color">#FF999999</color>
+    <color name="vector_fuchsia_color">#FFF56679</color>
+
 </resources>
diff --git a/app/src/main/res/values/themes_base.xml b/app/src/main/res/values/themes_base.xml
index 04ea854aaf..257f8fc527 100644
--- a/app/src/main/res/values/themes_base.xml
+++ b/app/src/main/res/values/themes_base.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
 
-    <style name="Base.V1.Theme.Riot" parent="Theme.AppCompat.Light.NoActionBar">
+    <style name="Base.V1.Theme.Riot" parent="Theme.MaterialComponents.Light.NoActionBar.Bridge">
         <item name="colorPrimary">@color/dark</item>
         <item name="colorPrimaryDark">@color/dark</item>
         <item name="colorAccent">@color/pale_teal</item>
@@ -11,5 +11,4 @@
     <style name="Base.Theme.Riot" parent="Base.V1.Theme.Riot" />
 
 
-
 </resources>
\ No newline at end of file
diff --git a/app/src/main/res/xml/pill_view.xml b/app/src/main/res/xml/pill_view.xml
new file mode 100644
index 0000000000..f51c9305b8
--- /dev/null
+++ b/app/src/main/res/xml/pill_view.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<chip xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    style="@style/Widget.MaterialComponents.Chip.Entry"
+    android:checkable="false"
+    app:closeIcon="@null" />
\ No newline at end of file

From 6c7bc2b40c3688264e824f2e70318d9127ffa312 Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Mon, 25 Feb 2019 18:10:37 +0100
Subject: [PATCH 04/13] Timeline: get better perf

---
 .../riotredesign/core/extensions/Event.kt     |  7 ++--
 .../core/resources/DateProvider.kt            | 32 +++++++++++++++++++
 .../features/home/HomeNavigator.kt            | 14 ++++----
 .../timeline/TimelineEventController.kt       |  3 --
 .../room/members/RoomMemberExtractor.kt       | 11 ++++---
 5 files changed, 48 insertions(+), 19 deletions(-)
 create mode 100644 app/src/main/java/im/vector/riotredesign/core/resources/DateProvider.kt

diff --git a/app/src/main/java/im/vector/riotredesign/core/extensions/Event.kt b/app/src/main/java/im/vector/riotredesign/core/extensions/Event.kt
index 098d1af576..d93877a6bd 100644
--- a/app/src/main/java/im/vector/riotredesign/core/extensions/Event.kt
+++ b/app/src/main/java/im/vector/riotredesign/core/extensions/Event.kt
@@ -17,12 +17,9 @@
 package im.vector.riotredesign.core.extensions
 
 import im.vector.matrix.android.api.session.events.model.Event
-import org.threeten.bp.Instant
+import im.vector.riotredesign.core.resources.DateProvider
 import org.threeten.bp.LocalDateTime
-import org.threeten.bp.ZoneId
-
 
 fun Event.localDateTime(): LocalDateTime {
-    val instant = Instant.ofEpochMilli(originServerTs ?: 0)
-    return LocalDateTime.ofInstant(instant, ZoneId.systemDefault())
+    return DateProvider.toLocalDateTime(originServerTs)
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/core/resources/DateProvider.kt b/app/src/main/java/im/vector/riotredesign/core/resources/DateProvider.kt
new file mode 100644
index 0000000000..cc01bdf183
--- /dev/null
+++ b/app/src/main/java/im/vector/riotredesign/core/resources/DateProvider.kt
@@ -0,0 +1,32 @@
+/*
+ * Copyright 2019 New Vector Ltd
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package im.vector.riotredesign.core.resources
+
+import org.threeten.bp.Instant
+import org.threeten.bp.LocalDateTime
+import org.threeten.bp.ZoneId
+
+object DateProvider {
+
+    private val zoneId = ZoneId.systemDefault()
+
+    fun toLocalDateTime(timestamp: Long?): LocalDateTime {
+        val instant = Instant.ofEpochMilli(timestamp ?: 0)
+        return LocalDateTime.ofInstant(instant, zoneId)
+    }
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeNavigator.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeNavigator.kt
index 7ffe463bb1..4fb84ea10d 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeNavigator.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeNavigator.kt
@@ -30,13 +30,13 @@ class HomeNavigator {
 
     var activity: HomeActivity? = null
 
-    private var currentRoomId: String? = null
+    private var rootRoomId: String? = null
 
     fun openRoomDetail(roomId: String,
                        eventId: String?,
                        addToBackstack: Boolean = false) {
         Timber.v("Open room detail $roomId - $eventId - $addToBackstack")
-        if (!addToBackstack && isRoomOpened(roomId)) {
+        if (!addToBackstack && isRoot(roomId)) {
             return
         }
         activity?.let {
@@ -46,7 +46,7 @@ class HomeNavigator {
             if (addToBackstack) {
                 it.addFragmentToBackstack(roomDetailFragment, R.id.homeDetailFragmentContainer, roomId)
             } else {
-                currentRoomId = roomId
+                rootRoomId = roomId
                 clearBackStack(it.supportFragmentManager)
                 it.replaceFragment(roomDetailFragment, R.id.homeDetailFragmentContainer)
             }
@@ -61,9 +61,7 @@ class HomeNavigator {
         Timber.v("Open user detail $userId")
     }
 
-    fun isRoomOpened(roomId: String): Boolean {
-        return currentRoomId == roomId
-    }
+    // Private Methods *****************************************************************************
 
     private fun clearBackStack(fragmentManager: FragmentManager) {
         if (fragmentManager.backStackEntryCount > 0) {
@@ -72,4 +70,8 @@ class HomeNavigator {
         }
     }
 
+    private fun isRoot(roomId: String): Boolean {
+        return rootRoomId == roomId
+    }
+
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
index 0d12ae8e45..e610931c48 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
@@ -37,9 +37,6 @@ class TimelineEventController(private val roomId: String,
         EpoxyAsyncUtil.getAsyncBackgroundHandler(),
         EpoxyAsyncUtil.getAsyncBackgroundHandler()
 ) {
-    init {
-        setFilterDuplicates(true)
-    }
 
     private var isLoadingForward: Boolean = false
     private var isLoadingBackward: Boolean = false
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMemberExtractor.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMemberExtractor.kt
index 5ab8728506..79d4a17686 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMemberExtractor.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMemberExtractor.kt
@@ -34,22 +34,23 @@ internal class RoomMemberExtractor(private val monarchy: Monarchy,
     private val cached = HashMap<String, RoomMember?>()
 
     fun extractFrom(event: EventEntity): RoomMember? {
-        if (cached.containsKey(event.eventId)) {
-            return cached[event.eventId]
-        }
         val sender = event.sender ?: return null
+        val cacheKey = sender + event.stateIndex
+        if (cached.containsKey(cacheKey)) {
+            return cached[cacheKey]
+        }
         // If the event is unlinked we want to fetch unlinked state events
         val unlinked = event.isUnlinked
         // When stateIndex is negative, we try to get the next stateEvent prevContent()
         // If prevContent is null we fallback to the Int.MIN state events content()
         val content = if (event.stateIndex <= 0) {
             baseQuery(monarchy, roomId, sender, unlinked).next(from = event.stateIndex)?.prevContent
-                    ?: baseQuery(monarchy, roomId, sender, unlinked).last(since = event.stateIndex)?.content
+            ?: baseQuery(monarchy, roomId, sender, unlinked).last(since = event.stateIndex)?.content
         } else {
             baseQuery(monarchy, roomId, sender, unlinked).last(since = event.stateIndex)?.content
         }
         val roomMember: RoomMember? = ContentMapper.map(content).toModel()
-        cached[event.eventId] = roomMember
+        cached[cacheKey] = roomMember
         return roomMember
     }
 

From 85b119bdcbd6dd65f998a56e401279439543ec5f Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Mon, 25 Feb 2019 18:13:53 +0100
Subject: [PATCH 05/13] Home : hide keyboard when drawer state change

---
 .../im/vector/riotredesign/core/extensions/Activity.kt |  4 ++++
 .../vector/riotredesign/features/home/HomeActivity.kt  | 10 ++++++++++
 2 files changed, 14 insertions(+)

diff --git a/app/src/main/java/im/vector/riotredesign/core/extensions/Activity.kt b/app/src/main/java/im/vector/riotredesign/core/extensions/Activity.kt
index e908974ebe..8c23d795c2 100644
--- a/app/src/main/java/im/vector/riotredesign/core/extensions/Activity.kt
+++ b/app/src/main/java/im/vector/riotredesign/core/extensions/Activity.kt
@@ -29,4 +29,8 @@ fun AppCompatActivity.replaceFragment(fragment: Fragment, frameId: Int) {
 
 fun AppCompatActivity.addFragmentToBackstack(fragment: Fragment, frameId: Int, tag: String? = null) {
     supportFragmentManager.inTransaction { replace(frameId, fragment).addToBackStack(tag) }
+}
+
+fun AppCompatActivity.hideKeyboard() {
+    currentFocus?.hideKeyboard()
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
index 7f1e00d3ad..12842034f8 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeActivity.kt
@@ -24,9 +24,11 @@ import android.view.MenuItem
 import androidx.appcompat.app.ActionBarDrawerToggle
 import androidx.appcompat.widget.Toolbar
 import androidx.core.view.GravityCompat
+import androidx.drawerlayout.widget.DrawerLayout
 import androidx.fragment.app.FragmentManager
 import com.airbnb.mvrx.viewModel
 import im.vector.riotredesign.R
+import im.vector.riotredesign.core.extensions.hideKeyboard
 import im.vector.riotredesign.core.extensions.observeEvent
 import im.vector.riotredesign.core.extensions.replaceFragment
 import im.vector.riotredesign.core.platform.OnBackPressed
@@ -44,11 +46,18 @@ class HomeActivity : RiotActivity(), ToolbarConfigurable {
     private val homeActivityViewModel: HomeActivityViewModel by viewModel()
     private val homeNavigator by inject<HomeNavigator>()
 
+    private val drawerListener = object : DrawerLayout.SimpleDrawerListener() {
+        override fun onDrawerStateChanged(newState: Int) {
+            hideKeyboard()
+        }
+    }
+
     override fun onCreate(savedInstanceState: Bundle?) {
         loadKoinModules(listOf(HomeModule(this).definition))
         homeNavigator.activity = this
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_home)
+        drawerLayout.addDrawerListener(drawerListener)
         if (savedInstanceState == null) {
             val homeDrawerFragment = HomeDrawerFragment.newInstance()
             val loadingDetail = LoadingRoomDetailFragment.newInstance()
@@ -61,6 +70,7 @@ class HomeActivity : RiotActivity(), ToolbarConfigurable {
     }
 
     override fun onDestroy() {
+        drawerLayout.removeDrawerListener(drawerListener)
         homeNavigator.activity = null
         super.onDestroy()
     }

From 40f1fcab18f35c7eeaa4c0584d9cf0a8fde09073 Mon Sep 17 00:00:00 2001
From: ganfra <francois.ganard@gmail.com>
Date: Mon, 25 Feb 2019 21:55:33 +0100
Subject: [PATCH 06/13] User : start handling users. Need to fine a good
 strategy to process room member events.

---
 .../matrix/android/api/session/Session.kt     |  3 +-
 .../android/api/session/user/UserService.kt   | 35 +++++++++++
 .../android/api/session/user/model/User.kt    | 29 +++++++++
 .../internal/database/model/UserEntity.kt     | 31 ++++++++++
 .../database/query/EventEntityQueries.kt      |  3 +-
 .../database/query/UserEntityQueries.kt       | 31 ++++++++++
 .../internal/session/DefaultSession.kt        | 12 +++-
 .../android/internal/session/SessionModule.kt |  7 ++-
 .../session/user/DefaultUserService.kt        | 40 ++++++++++++
 .../session/user/UserEntityUpdater.kt         | 61 +++++++++++++++++++
 10 files changed, 247 insertions(+), 5 deletions(-)
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/UserService.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/model/User.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/model/UserEntity.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/UserEntityQueries.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/DefaultUserService.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt

diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/Session.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/Session.kt
index 5d760a16f0..ad45328206 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/Session.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/Session.kt
@@ -21,12 +21,13 @@ import im.vector.matrix.android.api.auth.data.SessionParams
 import im.vector.matrix.android.api.session.content.ContentUrlResolver
 import im.vector.matrix.android.api.session.group.GroupService
 import im.vector.matrix.android.api.session.room.RoomService
+import im.vector.matrix.android.api.session.user.UserService
 
 /**
  * This interface defines interactions with a session.
  * An instance of a session will be provided by the SDK.
  */
-interface Session : RoomService, GroupService {
+interface Session : RoomService, GroupService, UserService {
 
     /**
      * The params associated to the session
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/UserService.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/UserService.kt
new file mode 100644
index 0000000000..8e90e5e970
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/UserService.kt
@@ -0,0 +1,35 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.api.session.user
+
+import im.vector.matrix.android.api.session.user.model.User
+
+/**
+ * This interface defines methods to get users. It's implemented at the session level.
+ */
+interface UserService {
+
+    /**
+     * Get a user from a userId
+     * @param userId the userId to look for.
+     * @return a user with userId or null
+     */
+    fun getUser(userId: String): User?
+
+}
\ No newline at end of file
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/model/User.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/model/User.kt
new file mode 100644
index 0000000000..14ed96d6b2
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/api/session/user/model/User.kt
@@ -0,0 +1,29 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.api.session.user.model
+
+/**
+ * Data class which holds information about a user.
+ * It can be retrieved with [im.vector.matrix.android.api.session.user.UserService]
+ */
+data class User(
+        val userId: String,
+        val displayName: String? = null,
+        val avatarUrl: String? = null
+)
\ No newline at end of file
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/model/UserEntity.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/model/UserEntity.kt
new file mode 100644
index 0000000000..af68d85601
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/model/UserEntity.kt
@@ -0,0 +1,31 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.internal.database.model
+
+import io.realm.RealmObject
+import io.realm.annotations.PrimaryKey
+
+internal open class UserEntity(@PrimaryKey var userId: String = "",
+                               var displayName: String = "",
+                               var avatarUrl: String = ""
+) : RealmObject() {
+
+    companion object
+}
+
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/EventEntityQueries.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/EventEntityQueries.kt
index 015a47acfe..4aa9088429 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/EventEntityQueries.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/EventEntityQueries.kt
@@ -27,8 +27,7 @@ import io.realm.Sort
 import io.realm.kotlin.where
 
 internal fun EventEntity.Companion.where(realm: Realm, eventId: String): RealmQuery<EventEntity> {
-    return realm.where<EventEntity>()
-            .equalTo(EventEntityFields.EVENT_ID, eventId)
+    return realm.where<EventEntity>().equalTo(EventEntityFields.EVENT_ID, eventId)
 }
 
 internal fun EventEntity.Companion.where(realm: Realm,
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/UserEntityQueries.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/UserEntityQueries.kt
new file mode 100644
index 0000000000..a7183dc279
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/UserEntityQueries.kt
@@ -0,0 +1,31 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.internal.database.query
+
+import im.vector.matrix.android.internal.database.model.UserEntity
+import im.vector.matrix.android.internal.database.model.UserEntityFields
+import io.realm.Realm
+import io.realm.RealmQuery
+import io.realm.kotlin.where
+
+internal fun UserEntity.Companion.where(realm: Realm, userId: String): RealmQuery<UserEntity> {
+    return realm
+            .where<UserEntity>()
+            .equalTo(UserEntityFields.USER_ID, userId)
+}
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
index 55b09aafe2..e04af0118d 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
@@ -28,6 +28,8 @@ import im.vector.matrix.android.api.session.group.model.GroupSummary
 import im.vector.matrix.android.api.session.room.Room
 import im.vector.matrix.android.api.session.room.RoomService
 import im.vector.matrix.android.api.session.room.model.RoomSummary
+import im.vector.matrix.android.api.session.user.UserService
+import im.vector.matrix.android.api.session.user.model.User
 import im.vector.matrix.android.internal.database.LiveEntityObserver
 import im.vector.matrix.android.internal.di.MatrixKoinComponent
 import im.vector.matrix.android.internal.di.MatrixKoinHolder
@@ -39,7 +41,7 @@ import org.koin.core.scope.Scope
 import org.koin.standalone.inject
 
 
-internal class DefaultSession(override val sessionParams: SessionParams) : Session, MatrixKoinComponent, RoomService {
+internal class DefaultSession(override val sessionParams: SessionParams) : Session, MatrixKoinComponent {
 
     companion object {
         const val SCOPE: String = "session"
@@ -51,6 +53,7 @@ internal class DefaultSession(override val sessionParams: SessionParams) : Sessi
     private val sessionListeners by inject<SessionListeners>()
     private val roomService by inject<RoomService>()
     private val groupService by inject<GroupService>()
+    private val userService by inject<UserService>()
     private val syncThread by inject<SyncThread>()
     private val contentUrlResolver by inject<ContentUrlResolver>()
     private var isOpen = false
@@ -118,6 +121,13 @@ internal class DefaultSession(override val sessionParams: SessionParams) : Sessi
         return groupService.liveGroupSummaries()
     }
 
+    // USER SERVICE
+
+    override fun getUser(userId: String): User? {
+        assert(isOpen)
+        return userService.getUser(userId)
+    }
+
     // Private methods *****************************************************************************
 
     private fun assertMainThread() {
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
index 91da7dd547..d12a232483 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
@@ -22,17 +22,18 @@ import im.vector.matrix.android.api.auth.data.SessionParams
 import im.vector.matrix.android.api.session.content.ContentUrlResolver
 import im.vector.matrix.android.api.session.group.GroupService
 import im.vector.matrix.android.api.session.room.RoomService
+import im.vector.matrix.android.api.session.user.UserService
 import im.vector.matrix.android.internal.database.LiveEntityObserver
 import im.vector.matrix.android.internal.session.content.DefaultContentUrlResolver
 import im.vector.matrix.android.internal.session.group.DefaultGroupService
 import im.vector.matrix.android.internal.session.group.GroupSummaryUpdater
 import im.vector.matrix.android.internal.session.room.DefaultRoomService
 import im.vector.matrix.android.internal.session.room.RoomAvatarResolver
-import im.vector.matrix.android.internal.session.room.RoomFactory
 import im.vector.matrix.android.internal.session.room.RoomSummaryUpdater
 import im.vector.matrix.android.internal.session.room.members.RoomDisplayNameResolver
 import im.vector.matrix.android.internal.session.room.members.RoomMemberDisplayNameResolver
 import im.vector.matrix.android.internal.session.room.prune.EventsPruner
+import im.vector.matrix.android.internal.session.user.DefaultUserService
 import im.vector.matrix.android.internal.util.md5
 import io.realm.RealmConfiguration
 import org.koin.dsl.module.module
@@ -96,6 +97,10 @@ internal class SessionModule(private val sessionParams: SessionParams) {
             DefaultGroupService(get()) as GroupService
         }
 
+        scope(DefaultSession.SCOPE) {
+            DefaultUserService(get()) as UserService
+        }
+
         scope(DefaultSession.SCOPE) {
             SessionListeners()
         }
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/DefaultUserService.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/DefaultUserService.kt
new file mode 100644
index 0000000000..b8e1f028bb
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/DefaultUserService.kt
@@ -0,0 +1,40 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.internal.session.user
+
+import com.zhuinden.monarchy.Monarchy
+import im.vector.matrix.android.api.session.user.UserService
+import im.vector.matrix.android.api.session.user.model.User
+import im.vector.matrix.android.internal.database.model.UserEntity
+import im.vector.matrix.android.internal.database.query.where
+import im.vector.matrix.android.internal.util.fetchCopied
+
+internal class DefaultUserService(private val monarchy: Monarchy) : UserService {
+
+    override fun getUser(userId: String): User? {
+        val userEntity = monarchy.fetchCopied { UserEntity.where(it, userId).findFirst() }
+                ?: return null
+
+        return User(
+                userEntity.userId,
+                userEntity.displayName,
+                userEntity.avatarUrl
+        )
+    }
+}
\ No newline at end of file
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt
new file mode 100644
index 0000000000..6360f5b8d2
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt
@@ -0,0 +1,61 @@
+/*
+ *
+ *  * Copyright 2019 New Vector Ltd
+ *  *
+ *  * Licensed under the Apache License, Version 2.0 (the "License");
+ *  * you may not use this file except in compliance with the License.
+ *  * You may obtain a copy of the License at
+ *  *
+ *  *     http://www.apache.org/licenses/LICENSE-2.0
+ *  *
+ *  * Unless required by applicable law or agreed to in writing, software
+ *  * distributed under the License is distributed on an "AS IS" BASIS,
+ *  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  * See the License for the specific language governing permissions and
+ *  * limitations under the License.
+ *
+ */
+
+package im.vector.matrix.android.internal.session.user
+
+import com.zhuinden.monarchy.Monarchy
+import im.vector.matrix.android.api.session.events.model.EventType
+import im.vector.matrix.android.internal.database.RealmLiveEntityObserver
+import im.vector.matrix.android.internal.database.mapper.asDomain
+import im.vector.matrix.android.internal.database.model.EventEntity
+import im.vector.matrix.android.internal.database.model.EventEntityFields
+import im.vector.matrix.android.internal.database.model.UserEntity
+import im.vector.matrix.android.internal.database.query.where
+import im.vector.matrix.android.internal.session.room.members.RoomMembers
+import io.realm.Realm
+import io.realm.Sort
+
+internal class UserEntityUpdater(monarchy: Monarchy)
+    : RealmLiveEntityObserver<EventEntity>(monarchy) {
+
+    override val query = Monarchy.Query<EventEntity> {
+        EventEntity.where(it, type = EventType.STATE_ROOM_MEMBER)
+                .sort(EventEntityFields.STATE_INDEX, Sort.DESCENDING)
+                .distinct(EventEntityFields.STATE_KEY)
+    }
+
+    override fun process(inserted: List<EventEntity>, updated: List<EventEntity>, deleted: List<EventEntity>) {
+        val roomMembersEvents = (inserted + updated).map { it.eventId }
+        monarchy.writeAsync { realm ->
+            roomMembersEvents.forEach { updateUser(realm, it) }
+        }
+    }
+
+    private fun updateUser(realm: Realm, eventId: String) {
+        val event = EventEntity.where(realm, eventId).findFirst()?.asDomain() ?: return
+        val roomId = event.roomId ?: return
+        val userId = event.stateKey ?: return
+        val roomMember = RoomMembers(realm, roomId).getLoaded()[userId] ?: return
+
+        val userEntity = UserEntity.where(realm, userId).findFirst()
+                ?: realm.createObject(UserEntity::class.java, userId)
+        userEntity.displayName = roomMember.displayName ?: ""
+        userEntity.avatarUrl = roomMember.avatarUrl ?: ""
+    }
+
+}
\ No newline at end of file

From f59977f88494bb707f0e1e285f16b018428cd930 Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Tue, 26 Feb 2019 14:50:53 +0100
Subject: [PATCH 07/13] Room state: fix sync state events duplicates with
 timeline

---
 .../android/internal/database/helper/RoomEntityHelper.kt    | 6 ++++--
 .../android/internal/database/query/RoomEntityQueries.kt    | 5 +++++
 .../matrix/android/internal/session/sync/RoomSyncHandler.kt | 2 +-
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/helper/RoomEntityHelper.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/helper/RoomEntityHelper.kt
index ada074791d..cec664c2f9 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/helper/RoomEntityHelper.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/helper/RoomEntityHelper.kt
@@ -21,6 +21,7 @@ import im.vector.matrix.android.internal.database.mapper.toEntity
 import im.vector.matrix.android.internal.database.mapper.updateWith
 import im.vector.matrix.android.internal.database.model.ChunkEntity
 import im.vector.matrix.android.internal.database.model.RoomEntity
+import im.vector.matrix.android.internal.database.query.fastContains
 
 
 internal fun RoomEntity.deleteOnCascade(chunkEntity: ChunkEntity) {
@@ -37,16 +38,17 @@ internal fun RoomEntity.addOrUpdate(chunkEntity: ChunkEntity) {
 
 internal fun RoomEntity.addStateEvents(stateEvents: List<Event>,
                                        stateIndex: Int = Int.MIN_VALUE,
+                                       filterDuplicates: Boolean = false,
                                        isUnlinked: Boolean = false) {
     if (!isManaged) {
         throw IllegalStateException("Chunk entity should be managed to use fast contains")
     }
     stateEvents.forEach { event ->
-        if (event.eventId == null) {
+        if (event.eventId == null || (filterDuplicates && fastContains(event.eventId))) {
             return@forEach
         }
         val eventEntity = event.toEntity(roomId)
         eventEntity.updateWith(stateIndex, isUnlinked)
         untimelinedStateEvents.add(eventEntity)
     }
-}
\ No newline at end of file
+}
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/RoomEntityQueries.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/RoomEntityQueries.kt
index 17c9246e10..211421ef56 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/RoomEntityQueries.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/query/RoomEntityQueries.kt
@@ -17,6 +17,7 @@
 package im.vector.matrix.android.internal.database.query
 
 import im.vector.matrix.android.api.session.room.model.MyMembership
+import im.vector.matrix.android.internal.database.model.EventEntity
 import im.vector.matrix.android.internal.database.model.RoomEntity
 import im.vector.matrix.android.internal.database.model.RoomEntityFields
 import io.realm.Realm
@@ -34,3 +35,7 @@ internal fun RoomEntity.Companion.where(realm: Realm, membership: MyMembership?
     }
     return query
 }
+
+internal fun RoomEntity.fastContains(eventId: String): Boolean {
+    return EventEntity.where(realm, eventId = eventId).findFirst() != null
+}
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/sync/RoomSyncHandler.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/sync/RoomSyncHandler.kt
index 7cd703c77c..4e69992568 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/sync/RoomSyncHandler.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/sync/RoomSyncHandler.kt
@@ -92,7 +92,7 @@ internal class RoomSyncHandler(private val monarchy: Monarchy,
 
         if (roomSync.state != null && roomSync.state.events.isNotEmpty()) {
             val untimelinedStateIndex = if (isInitialSync) Int.MIN_VALUE else stateIndexOffset
-            roomEntity.addStateEvents(roomSync.state.events, stateIndex = untimelinedStateIndex)
+            roomEntity.addStateEvents(roomSync.state.events, filterDuplicates = true, stateIndex = untimelinedStateIndex)
         }
 
         if (roomSync.timeline != null && roomSync.timeline.events.isNotEmpty()) {

From 46226c4efc802b0bb458963a58a3119c83cda880 Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Tue, 26 Feb 2019 15:04:43 +0100
Subject: [PATCH 08/13] Avatar : skip '@' when name is userId

---
 .../core/extensions/CharSequence.kt           | 22 -------------------
 .../features/home/AvatarRenderer.kt           |  7 ++++--
 2 files changed, 5 insertions(+), 24 deletions(-)
 delete mode 100644 app/src/main/java/im/vector/riotredesign/core/extensions/CharSequence.kt

diff --git a/app/src/main/java/im/vector/riotredesign/core/extensions/CharSequence.kt b/app/src/main/java/im/vector/riotredesign/core/extensions/CharSequence.kt
deleted file mode 100644
index b16c110a93..0000000000
--- a/app/src/main/java/im/vector/riotredesign/core/extensions/CharSequence.kt
+++ /dev/null
@@ -1,22 +0,0 @@
-/*
- * Copyright 2019 New Vector Ltd
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package im.vector.riotredesign.core.extensions
-
-
-fun CharSequence.firstCharAsString(): String {
-    return if (isNotEmpty()) this[0].toString() else ""
-}
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
index 5e66ddba27..ae21e6b916 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
@@ -21,10 +21,10 @@ import androidx.core.content.ContextCompat
 import com.amulyakhare.textdrawable.TextDrawable
 import com.bumptech.glide.request.RequestOptions
 import im.vector.matrix.android.api.Matrix
+import im.vector.matrix.android.api.MatrixPatterns
 import im.vector.matrix.android.api.session.room.model.RoomMember
 import im.vector.matrix.android.api.session.room.model.RoomSummary
 import im.vector.riotredesign.R
-import im.vector.riotredesign.core.extensions.firstCharAsString
 import im.vector.riotredesign.core.glide.GlideApp
 
 object AvatarRenderer {
@@ -43,7 +43,10 @@ object AvatarRenderer {
         }
         val resolvedUrl = Matrix.getInstance().currentSession.contentUrlResolver().resolveFullSize(avatarUrl)
         val avatarColor = ContextCompat.getColor(imageView.context, R.color.pale_teal)
-        val fallbackDrawable = TextDrawable.builder().buildRound(name.firstCharAsString().toUpperCase(), avatarColor)
+        val isNameUserId = MatrixPatterns.isUserId(name)
+        val firstLetterIndex = if (isNameUserId) 1 else 0
+        val firstLetter = name[firstLetterIndex].toString().toUpperCase()
+        val fallbackDrawable = TextDrawable.builder().buildRound(firstLetter, avatarColor)
 
         GlideApp
                 .with(imageView)

From d6f6764b0c4ee7b6cb6620ec2cb0c7818aec226a Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Tue, 26 Feb 2019 19:28:39 +0100
Subject: [PATCH 09/13] Timeline : avoid epoxy to build on main thread first

---
 .../home/room/detail/timeline/TimelineEventController.kt    | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
index e610931c48..ec865e4b8b 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/TimelineEventController.kt
@@ -40,7 +40,11 @@ class TimelineEventController(private val roomId: String,
 
     private var isLoadingForward: Boolean = false
     private var isLoadingBackward: Boolean = false
-    private var hasReachedEnd: Boolean = false
+    private var hasReachedEnd: Boolean = true
+
+    init {
+        requestModelBuild()
+    }
 
     var callback: Callback? = null
 

From 41b06bca60e86836c7e7eb660b87ae9be6c8de88 Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Tue, 26 Feb 2019 19:32:01 +0100
Subject: [PATCH 10/13] User : rework UserEntityUpdater (and make others
 RealmLiveEntityObserver process on MonarchyThread instead of Main)

---
 .../database/RealmLiveEntityObserver.kt       | 40 +++++++------
 .../internal/session/DefaultSession.kt        | 12 +++-
 .../android/internal/session/SessionModule.kt |  5 +-
 .../internal/session/room/DefaultRoom.kt      |  2 +-
 .../session/room/members/RoomMembers.kt       | 13 ++++-
 .../internal/session/user/UpdateUserTask.kt   | 56 +++++++++++++++++++
 .../session/user/UserEntityUpdater.kt         | 38 +++++--------
 .../internal/session/user/UserModule.kt       | 31 ++++++++++
 8 files changed, 152 insertions(+), 45 deletions(-)
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UpdateUserTask.kt
 create mode 100644 matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserModule.kt

diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/RealmLiveEntityObserver.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/RealmLiveEntityObserver.kt
index 760c671525..5c66c4ede4 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/RealmLiveEntityObserver.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/database/RealmLiveEntityObserver.kt
@@ -16,11 +16,12 @@
 
 package im.vector.matrix.android.internal.database
 
-import androidx.lifecycle.LiveData
-import androidx.lifecycle.Observer
 import com.zhuinden.monarchy.Monarchy
+import io.realm.OrderedCollectionChangeSet
 import io.realm.RealmObject
+import io.realm.RealmResults
 import java.util.concurrent.atomic.AtomicBoolean
+import java.util.concurrent.atomic.AtomicReference
 
 internal interface LiveEntityObserver {
     fun start()
@@ -28,38 +29,41 @@ internal interface LiveEntityObserver {
 }
 
 internal abstract class RealmLiveEntityObserver<T : RealmObject>(protected val monarchy: Monarchy)
-    : Observer<Monarchy.ManagedChangeSet<T>>, LiveEntityObserver {
+    : LiveEntityObserver {
 
     protected abstract val query: Monarchy.Query<T>
     private val isStarted = AtomicBoolean(false)
-    private val liveResults: LiveData<Monarchy.ManagedChangeSet<T>> by lazy {
-        monarchy.findAllManagedWithChanges(query)
-    }
+    private lateinit var results: AtomicReference<RealmResults<T>>
 
     override fun start() {
         if (isStarted.compareAndSet(false, true)) {
-            liveResults.observeForever(this)
+            monarchy.postToMonarchyThread {
+                val queryResults = query.createQuery(it).findAll()
+                queryResults.addChangeListener { t, changeSet ->
+                    onChanged(t, changeSet)
+                }
+                results = AtomicReference(queryResults)
+            }
         }
     }
 
     override fun dispose() {
         if (isStarted.compareAndSet(true, false)) {
-            liveResults.removeObserver(this)
+            monarchy.postToMonarchyThread {
+                results.getAndSet(null).removeAllChangeListeners()
+            }
         }
     }
 
     // PRIVATE
 
-    override fun onChanged(changeSet: Monarchy.ManagedChangeSet<T>?) {
-        if (changeSet == null) {
-            return
-        }
-        val insertionIndexes = changeSet.orderedCollectionChangeSet.insertions
-        val updateIndexes = changeSet.orderedCollectionChangeSet.changes
-        val deletionIndexes = changeSet.orderedCollectionChangeSet.deletions
-        val inserted = changeSet.realmResults.filterIndexed { index, _ -> insertionIndexes.contains(index) }
-        val updated = changeSet.realmResults.filterIndexed { index, _ -> updateIndexes.contains(index) }
-        val deleted = changeSet.realmResults.filterIndexed { index, _ -> deletionIndexes.contains(index) }
+    private fun onChanged(realmResults: RealmResults<T>, changeSet: OrderedCollectionChangeSet) {
+        val insertionIndexes = changeSet.insertions
+        val updateIndexes = changeSet.changes
+        val deletionIndexes = changeSet.deletions
+        val inserted = realmResults.filterIndexed { index, _ -> insertionIndexes.contains(index) }
+        val updated = realmResults.filterIndexed { index, _ -> updateIndexes.contains(index) }
+        val deleted = realmResults.filterIndexed { index, _ -> deletionIndexes.contains(index) }
         process(inserted, updated, deleted)
     }
 
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
index e04af0118d..bace6740ce 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/DefaultSession.kt
@@ -19,6 +19,7 @@ package im.vector.matrix.android.internal.session
 import android.os.Looper
 import androidx.annotation.MainThread
 import androidx.lifecycle.LiveData
+import com.zhuinden.monarchy.Monarchy
 import im.vector.matrix.android.api.auth.data.SessionParams
 import im.vector.matrix.android.api.session.Session
 import im.vector.matrix.android.api.session.content.ContentUrlResolver
@@ -37,6 +38,7 @@ import im.vector.matrix.android.internal.session.group.GroupModule
 import im.vector.matrix.android.internal.session.room.RoomModule
 import im.vector.matrix.android.internal.session.sync.SyncModule
 import im.vector.matrix.android.internal.session.sync.job.SyncThread
+import im.vector.matrix.android.internal.session.user.UserModule
 import org.koin.core.scope.Scope
 import org.koin.standalone.inject
 
@@ -49,6 +51,7 @@ internal class DefaultSession(override val sessionParams: SessionParams) : Sessi
 
     private lateinit var scope: Scope
 
+    private val monarchy by inject<Monarchy>()
     private val liveEntityUpdaters by inject<List<LiveEntityObserver>>()
     private val sessionListeners by inject<SessionListeners>()
     private val roomService by inject<RoomService>()
@@ -67,8 +70,12 @@ internal class DefaultSession(override val sessionParams: SessionParams) : Sessi
         val syncModule = SyncModule().definition
         val roomModule = RoomModule().definition
         val groupModule = GroupModule().definition
-        MatrixKoinHolder.instance.loadModules(listOf(sessionModule, syncModule, roomModule, groupModule))
+        val userModule = UserModule().definition
+        MatrixKoinHolder.instance.loadModules(listOf(sessionModule, syncModule, roomModule, groupModule, userModule))
         scope = getKoin().getOrCreateScope(SCOPE)
+        if (!monarchy.isMonarchyThreadOpen) {
+            monarchy.openManually()
+        }
         liveEntityUpdaters.forEach { it.start() }
         syncThread.start()
     }
@@ -80,6 +87,9 @@ internal class DefaultSession(override val sessionParams: SessionParams) : Sessi
         assert(isOpen)
         syncThread.kill()
         liveEntityUpdaters.forEach { it.dispose() }
+        if (monarchy.isMonarchyThreadOpen) {
+            monarchy.closeManually()
+        }
         scope.close()
         isOpen = false
     }
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
index d12a232483..5bbbc6cb30 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/SessionModule.kt
@@ -34,6 +34,8 @@ import im.vector.matrix.android.internal.session.room.members.RoomDisplayNameRes
 import im.vector.matrix.android.internal.session.room.members.RoomMemberDisplayNameResolver
 import im.vector.matrix.android.internal.session.room.prune.EventsPruner
 import im.vector.matrix.android.internal.session.user.DefaultUserService
+import im.vector.matrix.android.internal.session.user.UserEntityUpdater
+import im.vector.matrix.android.internal.session.user.UserModule
 import im.vector.matrix.android.internal.util.md5
 import io.realm.RealmConfiguration
 import org.koin.dsl.module.module
@@ -113,7 +115,8 @@ internal class SessionModule(private val sessionParams: SessionParams) {
             val roomSummaryUpdater = RoomSummaryUpdater(get(), get(), get(), get(), sessionParams.credentials)
             val groupSummaryUpdater = GroupSummaryUpdater(get())
             val eventsPruner = EventsPruner(get())
-            listOf<LiveEntityObserver>(roomSummaryUpdater, groupSummaryUpdater, eventsPruner)
+            val userEntityUpdater = UserEntityUpdater(get(), get(), get())
+            listOf<LiveEntityObserver>(roomSummaryUpdater, groupSummaryUpdater, eventsPruner, userEntityUpdater)
         }
 
 
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoom.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoom.kt
index e241a9f6b9..de3de80105 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoom.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoom.kt
@@ -34,7 +34,7 @@ import im.vector.matrix.android.internal.session.room.members.LoadRoomMembersTas
 import im.vector.matrix.android.internal.task.TaskExecutor
 import im.vector.matrix.android.internal.task.configureWith
 
-internal data class DefaultRoom(
+internal class DefaultRoom(
         override val roomId: String,
         private val loadRoomMembersTask: LoadRoomMembersTask,
         private val monarchy: Monarchy,
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMembers.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMembers.kt
index fab1f60938..1bbb792db8 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMembers.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/members/RoomMembers.kt
@@ -26,6 +26,7 @@ import im.vector.matrix.android.internal.database.model.EventEntityFields
 import im.vector.matrix.android.internal.database.model.RoomSummaryEntity
 import im.vector.matrix.android.internal.database.query.where
 import io.realm.Realm
+import io.realm.Sort
 
 internal class RoomMembers(private val realm: Realm,
                            private val roomId: String
@@ -35,6 +36,17 @@ internal class RoomMembers(private val realm: Realm,
         RoomSummaryEntity.where(realm, roomId).findFirst()
     }
 
+    fun get(userId: String): RoomMember? {
+        return EventEntity
+                .where(realm, roomId, EventType.STATE_ROOM_MEMBER)
+                .sort(EventEntityFields.STATE_INDEX, Sort.DESCENDING)
+                .equalTo(EventEntityFields.STATE_KEY, userId)
+                .findFirst()
+                ?.let {
+                    it.asDomain().content?.toModel<RoomMember>()
+                }
+    }
+
     fun getLoaded(): Map<String, RoomMember> {
         return EventEntity
                 .where(realm, roomId, EventType.STATE_ROOM_MEMBER)
@@ -45,7 +57,6 @@ internal class RoomMembers(private val realm: Realm,
                 .mapValues { it.value.content.toModel<RoomMember>()!! }
     }
 
-
     fun getNumberOfJoinedMembers(): Int {
         return roomSummary?.joinedMembersCount
                ?: getLoaded().filterValues { it.membership == Membership.JOIN }.size
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UpdateUserTask.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UpdateUserTask.kt
new file mode 100644
index 0000000000..4197156d9d
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UpdateUserTask.kt
@@ -0,0 +1,56 @@
+/*
+ * Copyright 2019 New Vector Ltd
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package im.vector.matrix.android.internal.session.user
+
+import arrow.core.Try
+import com.zhuinden.monarchy.Monarchy
+import im.vector.matrix.android.api.session.room.model.Membership
+import im.vector.matrix.android.internal.database.mapper.asDomain
+import im.vector.matrix.android.internal.database.model.EventEntity
+import im.vector.matrix.android.internal.database.model.UserEntity
+import im.vector.matrix.android.internal.database.query.where
+import im.vector.matrix.android.internal.session.room.members.RoomMembers
+import im.vector.matrix.android.internal.task.Task
+import im.vector.matrix.android.internal.util.tryTransactionSync
+
+internal interface UpdateUserTask : Task<UpdateUserTask.Params, Unit> {
+
+    data class Params(val eventIds: List<String>)
+
+}
+
+internal class DefaultUpdateUserTask(private val monarchy: Monarchy) : UpdateUserTask {
+
+    override fun execute(params: UpdateUserTask.Params): Try<Unit> {
+        return monarchy.tryTransactionSync { realm ->
+            params.eventIds.forEach { eventId ->
+                val event = EventEntity.where(realm, eventId).findFirst()?.asDomain()
+                            ?: return@forEach
+                val roomId = event.roomId ?: return@forEach
+                val userId = event.stateKey ?: return@forEach
+                val roomMember = RoomMembers(realm, roomId).get(userId) ?: return@forEach
+                if (roomMember.membership != Membership.JOIN) return@forEach
+
+                val userEntity = UserEntity.where(realm, userId).findFirst()
+                                 ?: realm.createObject(UserEntity::class.java, userId)
+                userEntity.displayName = roomMember.displayName ?: ""
+                userEntity.avatarUrl = roomMember.avatarUrl ?: ""
+            }
+        }
+    }
+
+}
\ No newline at end of file
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt
index 6360f5b8d2..24529ee11b 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserEntityUpdater.kt
@@ -21,41 +21,33 @@ package im.vector.matrix.android.internal.session.user
 import com.zhuinden.monarchy.Monarchy
 import im.vector.matrix.android.api.session.events.model.EventType
 import im.vector.matrix.android.internal.database.RealmLiveEntityObserver
-import im.vector.matrix.android.internal.database.mapper.asDomain
 import im.vector.matrix.android.internal.database.model.EventEntity
 import im.vector.matrix.android.internal.database.model.EventEntityFields
-import im.vector.matrix.android.internal.database.model.UserEntity
 import im.vector.matrix.android.internal.database.query.where
-import im.vector.matrix.android.internal.session.room.members.RoomMembers
-import io.realm.Realm
+import im.vector.matrix.android.internal.task.TaskExecutor
+import im.vector.matrix.android.internal.task.TaskThread
+import im.vector.matrix.android.internal.task.configureWith
 import io.realm.Sort
 
-internal class UserEntityUpdater(monarchy: Monarchy)
+internal class UserEntityUpdater(monarchy: Monarchy,
+                                 private val updateUserTask: UpdateUserTask,
+                                 private val taskExecutor: TaskExecutor)
     : RealmLiveEntityObserver<EventEntity>(monarchy) {
 
     override val query = Monarchy.Query<EventEntity> {
-        EventEntity.where(it, type = EventType.STATE_ROOM_MEMBER)
+        EventEntity
+                .where(it, type = EventType.STATE_ROOM_MEMBER)
                 .sort(EventEntityFields.STATE_INDEX, Sort.DESCENDING)
                 .distinct(EventEntityFields.STATE_KEY)
+
     }
 
     override fun process(inserted: List<EventEntity>, updated: List<EventEntity>, deleted: List<EventEntity>) {
-        val roomMembersEvents = (inserted + updated).map { it.eventId }
-        monarchy.writeAsync { realm ->
-            roomMembersEvents.forEach { updateUser(realm, it) }
-        }
+        val roomMembersEvents = inserted.map { it.eventId }
+        val taskParams = UpdateUserTask.Params(roomMembersEvents)
+        updateUserTask
+                .configureWith(taskParams)
+                .executeOn(TaskThread.IO)
+                .executeBy(taskExecutor)
     }
-
-    private fun updateUser(realm: Realm, eventId: String) {
-        val event = EventEntity.where(realm, eventId).findFirst()?.asDomain() ?: return
-        val roomId = event.roomId ?: return
-        val userId = event.stateKey ?: return
-        val roomMember = RoomMembers(realm, roomId).getLoaded()[userId] ?: return
-
-        val userEntity = UserEntity.where(realm, userId).findFirst()
-                ?: realm.createObject(UserEntity::class.java, userId)
-        userEntity.displayName = roomMember.displayName ?: ""
-        userEntity.avatarUrl = roomMember.avatarUrl ?: ""
-    }
-
 }
\ No newline at end of file
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserModule.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserModule.kt
new file mode 100644
index 0000000000..a82fddc8de
--- /dev/null
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/user/UserModule.kt
@@ -0,0 +1,31 @@
+/*
+ * Copyright 2019 New Vector Ltd
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package im.vector.matrix.android.internal.session.user
+
+import im.vector.matrix.android.internal.session.DefaultSession
+import org.koin.dsl.module.module
+
+internal class UserModule {
+
+    val definition = module(override = true) {
+
+        scope(DefaultSession.SCOPE) {
+            DefaultUpdateUserTask(get()) as UpdateUserTask
+        }
+
+    }
+}

From 63bf4355b9ed59cf2a46b6f88a1c081da4b7ed4a Mon Sep 17 00:00:00 2001
From: ganfra <francoisg@matrix.org>
Date: Tue, 26 Feb 2019 19:35:47 +0100
Subject: [PATCH 11/13] Html : continue work on Pills. Still need to find how
 to handle avatar drawable.

---
 .../features/home/AvatarRenderer.kt           | 48 +++++++++++++--
 .../riotredesign/features/home/HomeModule.kt  | 20 +++++-
 .../detail/timeline/MessageItemFactory.kt     |  2 +-
 .../{markdown => html}/EventHtmlRenderer.kt   | 30 +++++----
 .../features/html/PillDrawableFactory.kt      | 61 +++++++++++++++++++
 5 files changed, 136 insertions(+), 25 deletions(-)
 rename app/src/main/java/im/vector/riotredesign/features/{markdown => html}/EventHtmlRenderer.kt (86%)
 create mode 100644 app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt

diff --git a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
index ae21e6b916..c2e4e6a509 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
@@ -16,16 +16,21 @@
 
 package im.vector.riotredesign.features.home
 
+import android.content.Context
+import android.graphics.drawable.Drawable
 import android.widget.ImageView
 import androidx.core.content.ContextCompat
 import com.amulyakhare.textdrawable.TextDrawable
 import com.bumptech.glide.request.RequestOptions
+import com.bumptech.glide.request.target.SimpleTarget
+import com.bumptech.glide.request.transition.Transition
 import im.vector.matrix.android.api.Matrix
 import im.vector.matrix.android.api.MatrixPatterns
 import im.vector.matrix.android.api.session.room.model.RoomMember
 import im.vector.matrix.android.api.session.room.model.RoomSummary
 import im.vector.riotredesign.R
 import im.vector.riotredesign.core.glide.GlideApp
+import im.vector.riotredesign.core.glide.GlideRequest
 
 object AvatarRenderer {
 
@@ -41,19 +46,52 @@ object AvatarRenderer {
         if (name.isNullOrEmpty()) {
             return
         }
+        buildGlideRequest(imageView.context, name, avatarUrl).into(imageView)
+    }
+
+    fun load(context: Context, avatarUrl: String?, name: String?, callback: Callback) {
+        if (name.isNullOrEmpty()) {
+            return
+        }
+        buildGlideRequest(context, name, avatarUrl).into(CallbackTarget(callback))
+    }
+
+    private fun buildGlideRequest(context: Context, name: String, avatarUrl: String?): GlideRequest<Drawable> {
         val resolvedUrl = Matrix.getInstance().currentSession.contentUrlResolver().resolveFullSize(avatarUrl)
-        val avatarColor = ContextCompat.getColor(imageView.context, R.color.pale_teal)
+        val avatarColor = ContextCompat.getColor(context, R.color.pale_teal)
         val isNameUserId = MatrixPatterns.isUserId(name)
         val firstLetterIndex = if (isNameUserId) 1 else 0
         val firstLetter = name[firstLetterIndex].toString().toUpperCase()
         val fallbackDrawable = TextDrawable.builder().buildRound(firstLetter, avatarColor)
-
-        GlideApp
-                .with(imageView)
+        return GlideApp
+                .with(context)
                 .load(resolvedUrl)
                 .placeholder(fallbackDrawable)
                 .apply(RequestOptions.circleCropTransform())
-                .into(imageView)
+    }
+
+    interface Callback {
+        fun onDrawableUpdated(drawable: Drawable?)
+        fun onDestroy()
+    }
+
+    private class CallbackTarget(private val callback: Callback) : SimpleTarget<Drawable>() {
+
+        override fun onDestroy() {
+            callback.onDestroy()
+        }
+
+        override fun onResourceReady(resource: Drawable, transition: Transition<in Drawable>?) {
+            callback.onDrawableUpdated(resource)
+        }
+
+        override fun onLoadStarted(placeholder: Drawable?) {
+            callback.onDrawableUpdated(placeholder)
+        }
+
+        override fun onLoadFailed(errorDrawable: Drawable?) {
+            callback.onDrawableUpdated(errorDrawable)
+        }
     }
 
 
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
index 35c8276cd8..2862765708 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/HomeModule.kt
@@ -16,25 +16,39 @@
 
 package im.vector.riotredesign.features.home
 
+import im.vector.matrix.android.api.Matrix
 import im.vector.riotredesign.features.home.group.SelectedGroupStore
 import im.vector.riotredesign.features.home.room.VisibleRoomStore
-import im.vector.riotredesign.features.home.room.detail.timeline.*
+import im.vector.riotredesign.features.home.room.detail.timeline.CallItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.DefaultItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.MessageItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.RoomHistoryVisibilityItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.RoomMemberItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.RoomNameItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.RoomTopicItemFactory
+import im.vector.riotredesign.features.home.room.detail.timeline.TimelineDateFormatter
+import im.vector.riotredesign.features.home.room.detail.timeline.TimelineEventController
+import im.vector.riotredesign.features.home.room.detail.timeline.TimelineItemFactory
 import im.vector.riotredesign.features.home.room.detail.timeline.helper.TimelineMediaSizeProvider
 import im.vector.riotredesign.features.home.room.list.RoomSummaryComparator
 import im.vector.riotredesign.features.home.room.list.RoomSummaryController
-import im.vector.riotredesign.features.markdown.EventHtmlRenderer
+import im.vector.riotredesign.features.html.EventHtmlRenderer
 import org.koin.dsl.module.module
 
 class HomeModule(homeActivity: HomeActivity) {
 
     val definition = module(override = true) {
 
+        single {
+            Matrix.getInstance().currentSession
+        }
+
         single {
             TimelineDateFormatter(get())
         }
 
         single {
-            EventHtmlRenderer(homeActivity)
+            EventHtmlRenderer(homeActivity, get())
         }
 
         single {
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
index e1eaaecc6f..d27475f77b 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/timeline/MessageItemFactory.kt
@@ -29,7 +29,7 @@ import im.vector.riotredesign.core.epoxy.RiotEpoxyModel
 import im.vector.riotredesign.core.extensions.localDateTime
 import im.vector.riotredesign.core.resources.ColorProvider
 import im.vector.riotredesign.features.home.room.detail.timeline.helper.TimelineMediaSizeProvider
-import im.vector.riotredesign.features.markdown.EventHtmlRenderer
+import im.vector.riotredesign.features.html.EventHtmlRenderer
 import im.vector.riotredesign.features.media.MediaContentRenderer
 import me.gujun.android.span.span
 
diff --git a/app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/html/EventHtmlRenderer.kt
similarity index 86%
rename from app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt
rename to app/src/main/java/im/vector/riotredesign/features/html/EventHtmlRenderer.kt
index 206d4282be..dcbe28f11b 100644
--- a/app/src/main/java/im/vector/riotredesign/features/markdown/EventHtmlRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/html/EventHtmlRenderer.kt
@@ -16,14 +16,13 @@
  *
  */
 
-package im.vector.riotredesign.features.markdown
+package im.vector.riotredesign.features.html
 
 import android.content.Context
 import android.text.style.ImageSpan
-import com.google.android.material.chip.ChipDrawable
 import im.vector.matrix.android.api.permalinks.PermalinkData
 import im.vector.matrix.android.api.permalinks.PermalinkParser
-import im.vector.riotredesign.R
+import im.vector.matrix.android.api.session.Session
 import org.commonmark.node.BlockQuote
 import org.commonmark.node.HtmlBlock
 import org.commonmark.node.HtmlInline
@@ -50,10 +49,11 @@ import ru.noties.markwon.html.tag.SuperScriptHandler
 import ru.noties.markwon.html.tag.UnderlineHandler
 import java.util.Arrays.asList
 
-class EventHtmlRenderer(private val context: Context) {
+class EventHtmlRenderer(private val context: Context,
+                        private val session: Session) {
 
     private val markwon = Markwon.builder(context)
-            .usePlugin(MatrixPlugin.create(context))
+            .usePlugin(MatrixPlugin.create(context, session))
             .build()
 
     fun render(text: String): CharSequence {
@@ -62,7 +62,8 @@ class EventHtmlRenderer(private val context: Context) {
 
 }
 
-private class MatrixPlugin private constructor(private val context: Context) : AbstractMarkwonPlugin() {
+private class MatrixPlugin private constructor(private val context: Context,
+                                               private val session: Session) : AbstractMarkwonPlugin() {
 
     override fun configureConfiguration(builder: MarkwonConfiguration.Builder) {
         builder.htmlParser(MarkwonHtmlParserImpl.create())
@@ -75,7 +76,7 @@ private class MatrixPlugin private constructor(private val context: Context) : A
                         ImageHandler.create())
                 .addHandler(
                         "a",
-                        MxLinkHandler(context))
+                        MxLinkHandler(context, session))
                 .addHandler(
                         "blockquote",
                         BlockquoteHandler())
@@ -127,13 +128,13 @@ private class MatrixPlugin private constructor(private val context: Context) : A
 
     companion object {
 
-        fun create(context: Context): MatrixPlugin {
-            return MatrixPlugin(context)
+        fun create(context: Context, session: Session): MatrixPlugin {
+            return MatrixPlugin(context, session)
         }
     }
 }
 
-private class MxLinkHandler(private val context: Context) : TagHandler() {
+private class MxLinkHandler(private val context: Context, private val session: Session) : TagHandler() {
 
     private val linkHandler = LinkHandler()
 
@@ -143,12 +144,9 @@ private class MxLinkHandler(private val context: Context) : TagHandler() {
             val permalinkData = PermalinkParser.parse(link)
             when (permalinkData) {
                 is PermalinkData.UserLink -> {
-                    val chipDrawable = ChipDrawable.createFromResource(context, R.xml.pill_view)
-                    chipDrawable.setText(permalinkData.userId)
-                    chipDrawable.textEndPadding = 8f
-                    chipDrawable.textStartPadding = 8f
-                    chipDrawable.setBounds(0, 0, chipDrawable.intrinsicWidth, (chipDrawable.intrinsicHeight / 1.5f).toInt())
-                    val span = ImageSpan(chipDrawable)
+                    val user = session.getUser(permalinkData.userId) ?: return
+                    val drawable = PillDrawableFactory.create(context, permalinkData.userId, user)
+                    val span = ImageSpan(drawable)
                     SpannableBuilder.setSpans(
                             visitor.builder(),
                             span,
diff --git a/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt b/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt
new file mode 100644
index 0000000000..05885491f1
--- /dev/null
+++ b/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt
@@ -0,0 +1,61 @@
+/*
+ * Copyright 2019 New Vector Ltd
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package im.vector.riotredesign.features.html
+
+import android.content.Context
+import android.graphics.drawable.Drawable
+import com.google.android.material.chip.ChipDrawable
+import im.vector.matrix.android.api.session.user.model.User
+import im.vector.riotredesign.R
+import im.vector.riotredesign.features.home.AvatarRenderer
+import java.lang.ref.WeakReference
+
+object PillDrawableFactory {
+
+    fun create(context: Context, userId: String, user: User?): Drawable {
+        val chipDrawable = ChipDrawable.createFromResource(context, R.xml.pill_view).apply {
+            setText(user?.displayName ?: userId)
+            textEndPadding = 8f
+            textStartPadding = 8f
+            setBounds(0, 0, intrinsicWidth, (intrinsicHeight / 1.5f).toInt())
+        }
+        val avatarRendererCallback = AvatarRendererChipCallback(chipDrawable)
+        // TODO: need to work on getting drawable async
+        //AvatarRenderer.load(context, user?.avatarUrl, user?.displayName, avatarRendererCallback)
+        return chipDrawable
+    }
+
+    private class AvatarRendererChipCallback(chipDrawable: ChipDrawable) : AvatarRenderer.Callback {
+
+        private val weakChipDrawable = WeakReference<ChipDrawable>(chipDrawable)
+
+        override fun onDestroy() {
+            weakChipDrawable.clear()
+        }
+
+        override fun onDrawableUpdated(drawable: Drawable?) {
+            weakChipDrawable.get()?.apply {
+                chipIcon = drawable
+                setBounds(0, 0, intrinsicWidth, (intrinsicHeight / 1.5f).toInt())
+            }
+        }
+
+    }
+
+
+}
+

From adc51529f224849ae01197fc1e3435b000b6e05e Mon Sep 17 00:00:00 2001
From: ganfra <francois.ganard@gmail.com>
Date: Wed, 27 Feb 2019 17:17:47 +0100
Subject: [PATCH 12/13] Pills : handle avatar

---
 app/build.gradle                              |  3 +
 .../features/home/AvatarRenderer.kt           | 60 +++++++++----------
 .../features/html/PillDrawableFactory.kt      | 20 +++----
 app/src/main/res/values/dimens.xml            |  7 +++
 4 files changed, 46 insertions(+), 44 deletions(-)
 create mode 100644 app/src/main/res/values/dimens.xml

diff --git a/app/build.gradle b/app/build.gradle
index e91f74d450..58cc5e064f 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -60,6 +60,7 @@ dependencies {
 
     def epoxy_version = "3.0.0"
     def arrow_version = "0.8.2"
+    def coroutines_version = "1.0.1"
     def markwon_version = '3.0.0-SNAPSHOT'
 
     implementation project(":matrix-sdk-android")
@@ -67,6 +68,8 @@ dependencies {
     implementation 'com.android.support:multidex:1.0.3'
 
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
+    implementation "org.jetbrains.kotlinx:kotlinx-coroutines-core:$coroutines_version"
+    implementation "org.jetbrains.kotlinx:kotlinx-coroutines-android:$coroutines_version"
 
     implementation 'androidx.appcompat:appcompat:1.1.0-alpha01'
     implementation 'androidx.constraintlayout:constraintlayout:1.1.3'
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
index c2e4e6a509..3227dc6502 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/AvatarRenderer.kt
@@ -22,8 +22,6 @@ import android.widget.ImageView
 import androidx.core.content.ContextCompat
 import com.amulyakhare.textdrawable.TextDrawable
 import com.bumptech.glide.request.RequestOptions
-import com.bumptech.glide.request.target.SimpleTarget
-import com.bumptech.glide.request.transition.Transition
 import im.vector.matrix.android.api.Matrix
 import im.vector.matrix.android.api.MatrixPatterns
 import im.vector.matrix.android.api.session.room.model.RoomMember
@@ -31,6 +29,8 @@ import im.vector.matrix.android.api.session.room.model.RoomSummary
 import im.vector.riotredesign.R
 import im.vector.riotredesign.core.glide.GlideApp
 import im.vector.riotredesign.core.glide.GlideRequest
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
 
 object AvatarRenderer {
 
@@ -46,53 +46,47 @@ object AvatarRenderer {
         if (name.isNullOrEmpty()) {
             return
         }
-        buildGlideRequest(imageView.context, name, avatarUrl).into(imageView)
+        val placeholder = buildPlaceholderDrawable(imageView.context, name)
+        buildGlideRequest(imageView.context, avatarUrl)
+                .placeholder(placeholder)
+                .into(imageView)
     }
 
-    fun load(context: Context, avatarUrl: String?, name: String?, callback: Callback) {
+    fun load(context: Context, avatarUrl: String?, name: String?, size: Int, callback: Callback) {
         if (name.isNullOrEmpty()) {
             return
         }
-        buildGlideRequest(context, name, avatarUrl).into(CallbackTarget(callback))
+        val request = buildGlideRequest(context, avatarUrl)
+        GlobalScope.launch {
+            val placeholder = buildPlaceholderDrawable(context, name)
+            callback.onDrawableUpdated(placeholder)
+            try {
+                val drawable = request.submit(size, size).get()
+                callback.onDrawableUpdated(drawable)
+            } catch (exception: Exception) {
+                callback.onDrawableUpdated(placeholder)
+            }
+        }
     }
 
-    private fun buildGlideRequest(context: Context, name: String, avatarUrl: String?): GlideRequest<Drawable> {
+    private fun buildGlideRequest(context: Context, avatarUrl: String?): GlideRequest<Drawable> {
         val resolvedUrl = Matrix.getInstance().currentSession.contentUrlResolver().resolveFullSize(avatarUrl)
+        return GlideApp
+                .with(context)
+                .load(resolvedUrl)
+                .apply(RequestOptions.circleCropTransform())
+    }
+
+    private fun buildPlaceholderDrawable(context: Context, name: String): Drawable {
         val avatarColor = ContextCompat.getColor(context, R.color.pale_teal)
         val isNameUserId = MatrixPatterns.isUserId(name)
         val firstLetterIndex = if (isNameUserId) 1 else 0
         val firstLetter = name[firstLetterIndex].toString().toUpperCase()
-        val fallbackDrawable = TextDrawable.builder().buildRound(firstLetter, avatarColor)
-        return GlideApp
-                .with(context)
-                .load(resolvedUrl)
-                .placeholder(fallbackDrawable)
-                .apply(RequestOptions.circleCropTransform())
+        return TextDrawable.builder().buildRound(firstLetter, avatarColor)
     }
 
     interface Callback {
         fun onDrawableUpdated(drawable: Drawable?)
-        fun onDestroy()
     }
 
-    private class CallbackTarget(private val callback: Callback) : SimpleTarget<Drawable>() {
-
-        override fun onDestroy() {
-            callback.onDestroy()
-        }
-
-        override fun onResourceReady(resource: Drawable, transition: Transition<in Drawable>?) {
-            callback.onDrawableUpdated(resource)
-        }
-
-        override fun onLoadStarted(placeholder: Drawable?) {
-            callback.onDrawableUpdated(placeholder)
-        }
-
-        override fun onLoadFailed(errorDrawable: Drawable?) {
-            callback.onDrawableUpdated(errorDrawable)
-        }
-    }
-
-
 }
\ No newline at end of file
diff --git a/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt b/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt
index 05885491f1..9b87c67135 100644
--- a/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/html/PillDrawableFactory.kt
@@ -27,15 +27,18 @@ import java.lang.ref.WeakReference
 object PillDrawableFactory {
 
     fun create(context: Context, userId: String, user: User?): Drawable {
+        val textPadding = context.resources.getDimension(R.dimen.pill_text_padding)
+
         val chipDrawable = ChipDrawable.createFromResource(context, R.xml.pill_view).apply {
             setText(user?.displayName ?: userId)
-            textEndPadding = 8f
-            textStartPadding = 8f
-            setBounds(0, 0, intrinsicWidth, (intrinsicHeight / 1.5f).toInt())
+            textEndPadding = textPadding
+            textStartPadding = textPadding
+            setChipMinHeightResource(R.dimen.pill_min_height)
+            setChipIconSizeResource(R.dimen.pill_avatar_size)
+            setBounds(0, 0, intrinsicWidth, intrinsicHeight)
         }
         val avatarRendererCallback = AvatarRendererChipCallback(chipDrawable)
-        // TODO: need to work on getting drawable async
-        //AvatarRenderer.load(context, user?.avatarUrl, user?.displayName, avatarRendererCallback)
+        AvatarRenderer.load(context, user?.avatarUrl, user?.displayName, 80, avatarRendererCallback)
         return chipDrawable
     }
 
@@ -43,19 +46,14 @@ object PillDrawableFactory {
 
         private val weakChipDrawable = WeakReference<ChipDrawable>(chipDrawable)
 
-        override fun onDestroy() {
-            weakChipDrawable.clear()
-        }
-
         override fun onDrawableUpdated(drawable: Drawable?) {
             weakChipDrawable.get()?.apply {
                 chipIcon = drawable
-                setBounds(0, 0, intrinsicWidth, (intrinsicHeight / 1.5f).toInt())
+                setBounds(0, 0, intrinsicWidth, intrinsicHeight)
             }
         }
 
     }
 
-
 }
 
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
new file mode 100644
index 0000000000..3d9cae027d
--- /dev/null
+++ b/app/src/main/res/values/dimens.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<resources>
+    <dimen name="pill_avatar_size">16dp</dimen>
+    <dimen name="pill_min_height">20dp</dimen>
+    <dimen name="pill_text_padding">4dp</dimen>
+</resources>
\ No newline at end of file

From 2c0bc93f5a3023bbdcf6c7d1c50780f428af2c71 Mon Sep 17 00:00:00 2001
From: ganfra <francois.ganard@gmail.com>
Date: Wed, 27 Feb 2019 17:41:06 +0100
Subject: [PATCH 13/13] Home: getting Room interface slow (blocking Main
 Thread). Maybe we should get this async.

---
 .../session/room/DefaultRoomService.kt         |  4 ++--
 .../matrix/android/internal/util/Monarchy.kt   | 18 ++++++++++++++++--
 2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoomService.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoomService.kt
index b14627899f..0b0bae8da7 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoomService.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/room/DefaultRoomService.kt
@@ -26,13 +26,13 @@ import im.vector.matrix.android.internal.database.model.RoomEntity
 import im.vector.matrix.android.internal.database.model.RoomSummaryEntity
 import im.vector.matrix.android.internal.database.model.RoomSummaryEntityFields
 import im.vector.matrix.android.internal.database.query.where
-import im.vector.matrix.android.internal.util.fetchCopied
+import im.vector.matrix.android.internal.util.fetchManaged
 
 internal class DefaultRoomService(private val monarchy: Monarchy,
                                   private val roomFactory: RoomFactory) : RoomService {
 
     override fun getRoom(roomId: String): Room? {
-        monarchy.fetchCopied { RoomEntity.where(it, roomId).findFirst() } ?: return null
+        monarchy.fetchManaged { RoomEntity.where(it, roomId).findFirst() } ?: return null
         return roomFactory.instantiate(roomId)
     }
 
diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/util/Monarchy.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/util/Monarchy.kt
index 809ba16bf0..2a0fc5d708 100644
--- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/util/Monarchy.kt
+++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/util/Monarchy.kt
@@ -34,11 +34,25 @@ internal fun Monarchy.tryTransactionAsync(transaction: (realm: Realm) -> Unit):
     }
 }
 
+fun <T : RealmModel> Monarchy.fetchManaged(query: (Realm) -> T?): T? {
+    return fetch(query, false)
+}
+
 fun <T : RealmModel> Monarchy.fetchCopied(query: (Realm) -> T?): T? {
+    return fetch(query, true)
+}
+
+private fun <T : RealmModel> Monarchy.fetch(query: (Realm) -> T?, copyFromRealm: Boolean): T? {
     val ref = AtomicReference<T>()
     doWithRealm { realm ->
-        val result = query.invoke(realm)?.let { realm.copyFromRealm(it) }
+        val result = query.invoke(realm)?.let {
+            if (copyFromRealm) {
+                realm.copyFromRealm(it)
+            } else {
+                it
+            }
+        }
         ref.set(result)
     }
     return ref.get()
-}
\ No newline at end of file
+}