app/build.gradle

@@ -25,7 +25,6 @@ android {
         release {
             minifyEnabled false
             proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
-            signingConfig signingConfigs.debug
         }
     }
     compileOptions {
@@ -43,7 +42,6 @@ dependencies {
     implementation 'com.google.android.material:material:1.4.0'
     implementation 'androidx.constraintlayout:constraintlayout:2.1.2'
     implementation "androidx.lifecycle:lifecycle-viewmodel-ktx:2.4.0"
-    implementation 'androidx.legacy:legacy-support-v4:1.0.0'
     testImplementation 'junit:junit:4.13.2'
     androidTestImplementation 'androidx.test.ext:junit:1.1.3'
     androidTestImplementation 'androidx.test.espresso:espresso-core:3.4.0'

app/src/main/java/dev/lowrespalmtree/comet/ContentRecycler.kt

@@ -1,6 +1,5 @@
 package dev.lowrespalmtree.comet
 
-import android.annotation.SuppressLint
 import android.text.SpannableString
 import android.text.style.UnderlineSpan
 import android.util.Log
@@ -13,8 +12,6 @@ import dev.lowrespalmtree.comet.databinding.*
 class ContentAdapter(private var content: List<Line>, private val listener: ContentAdapterListen) :
     RecyclerView.Adapter<ContentAdapter.ContentViewHolder>() {
 
-    private var lastLineCount = 0
-
     interface ContentAdapterListen {
         fun onLinkClick(url: String)
     }
@@ -38,6 +35,7 @@ class ContentAdapter(private var content: List<Line>, private val listener: Cont
         }
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ContentViewHolder {
+        Log.d(TAG, "onCreateViewHolder: type $viewType")
         return LayoutInflater.from(parent.context).let {
             when (viewType) {
                 TYPE_EMPTY -> ContentViewHolder.Empty(GemtextEmptyBinding.inflate(it))
@@ -55,12 +53,12 @@ class ContentAdapter(private var content: List<Line>, private val listener: Cont
         }
     }
 
-    @SuppressLint("SetTextI18n")
     override fun onBindViewHolder(holder: ContentViewHolder, position: Int) {
+        Log.d(TAG, "onBindViewHolder: position $position")
         val line = content[position]
         when (holder) {
-            is ContentViewHolder.Paragraph ->
+            is ContentViewHolder.Paragraph -> holder.binding.textView.text =
-                holder.binding.textView.text = (line as ParagraphLine).text
+                (line as ParagraphLine).text
             is ContentViewHolder.Title1 -> holder.binding.textView.text = (line as TitleLine).text
             is ContentViewHolder.Title2 -> holder.binding.textView.text = (line as TitleLine).text
             is ContentViewHolder.Title3 -> holder.binding.textView.text = (line as TitleLine).text
@@ -71,40 +69,15 @@ class ContentAdapter(private var content: List<Line>, private val listener: Cont
                 holder.binding.textView.text = underlined
                 holder.binding.root.setOnClickListener { listener.onLinkClick(line.url) }
             }
-            is ContentViewHolder.PreText ->
-                holder.binding.textView.text = (line as PreTextLine).text
-            is ContentViewHolder.Blockquote ->
-                holder.binding.textView.text = (line as BlockquoteLine).text
-            is ContentViewHolder.ListItem ->
-                holder.binding.textView.text = "\u25CF ${(line as ListItemLine).text}"
             else -> {}
         }
     }
 
     override fun getItemCount(): Int = content.size
 
-    /**
+    fun setContent(content: List<Line>) {
-     * Replace the content rendered by the recycler.
+        this.content = content
-     *
-     * The new content list may or may not be the same object as the previous one, we don't
-     * assume anything. The assumptions this function do however are:
-     * - If the new content is empty, we are about to load new content, so clear the recycler.
-     * - If it's longer than before, we received new streamed content, so *append* data.
-     * - If it's shorter or the same size than before, we do not notify anything and let the caller
-     *   manage the changes itself.
-     */
-    @SuppressLint("NotifyDataSetChanged")
-    fun setContent(newContent: List<Line>) {
-        content = newContent.toList()  // Shallow copy to avoid concurrent update issues.
-        if (content.isEmpty()) {
-            Log.d(TAG, "setContent: empty content")
         notifyDataSetChanged()
-        } else if (content.size > lastLineCount) {
-            val numAdded = content.size - lastLineCount
-            Log.d(TAG, "setContent: added $numAdded items")
-            notifyItemRangeInserted(lastLineCount, numAdded)
-        }
-        lastLineCount = content.size
     }
 
     sealed class ContentViewHolder(view: View) : RecyclerView.ViewHolder(view) {

app/src/main/java/dev/lowrespalmtree/comet/Gemtext.kt

@@ -41,7 +41,6 @@ fun parseData(
                 channel.send(line)
             }
         }
-        channel.close()
     }
     return channel
 }

app/src/main/java/dev/lowrespalmtree/comet/MainActivity.kt

@@ -2,7 +2,6 @@ package dev.lowrespalmtree.comet
 
 import android.annotation.SuppressLint
 import android.app.Activity
-import android.content.ActivityNotFoundException
 import android.content.Intent
 import android.content.Intent.ACTION_VIEW
 import android.net.Uri
@@ -19,7 +18,6 @@ import androidx.lifecycle.viewModelScope
 import androidx.recyclerview.widget.LinearLayoutManager
 import dev.lowrespalmtree.comet.databinding.ActivityMainBinding
 import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.Job
 import kotlinx.coroutines.launch
 import java.net.UnknownHostException
 import java.nio.charset.Charset
@@ -29,8 +27,6 @@ class MainActivity : AppCompatActivity(), ContentAdapter.ContentAdapterListen {
     private lateinit var pageViewModel: PageViewModel
     private lateinit var adapter: ContentAdapter
 
-    private val currentUrl get() = binding.addressBar.text
-
     @SuppressLint("SetTextI18n")
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
@@ -54,11 +50,8 @@ class MainActivity : AppCompatActivity(), ContentAdapter.ContentAdapterListen {
             }
         }
 
-        binding.contentSwipeLayout.setOnRefreshListener { openUrl(currentUrl.toString()) }
+        pageViewModel.linesLiveData.observe(this, { adapter.setContent(it) })
-
+        pageViewModel.alertLiveData.observe(this, { alert(it) })
-        pageViewModel.state.observe(this, { updateState(it) })
-        pageViewModel.lines.observe(this, { updateLines(it) })
-        pageViewModel.alert.observe(this, { alert(it) })
     }
 
     override fun onLinkClick(url: String) {
@@ -70,68 +63,29 @@ class MainActivity : AppCompatActivity(), ContentAdapter.ContentAdapterListen {
         var uri = Uri.parse(url)
         if (!uri.isAbsolute) {
             uri = if (!base.isNullOrEmpty()) joinUrls(base, url) else toGeminiUri(uri)
+            Log.d(TAG, "openUrl: '$url' - '$base' - '$uri'")
+            Log.d(TAG, "openUrl: ${uri.authority} - ${uri.path} - ${uri.query}")
+            binding.addressBar.setText(uri.toString())
         }
 
         when (uri.scheme) {
-            "gemini" -> {
+            "gemini" -> pageViewModel.sendGeminiRequest(uri)
-                binding.addressBar.setText(uri.toString())
+            else -> startActivity(Intent(ACTION_VIEW, uri))
-                pageViewModel.sendGeminiRequest(uri)
-            }
-            else -> openUnknownScheme(uri)
         }
     }
 
-    private fun updateState(state: PageViewModel.State) {
+    private fun alert(message: String) {
-        Log.d(TAG, "updateState: $state")
+        AlertDialog.Builder(this)
-        when (state) {
+            .setTitle(R.string.alert_title)
-            PageViewModel.State.IDLE -> {
-                binding.contentProgressBar.hide()
-                binding.contentSwipeLayout.isRefreshing = false
-            }
-            PageViewModel.State.CONNECTING -> {
-                binding.contentProgressBar.show()
-            }
-            PageViewModel.State.RECEIVING -> {
-                binding.contentSwipeLayout.isRefreshing = false
-            }
-        }
-    }
-
-    private fun updateLines(lines: List<Line>) {
-        Log.d(TAG, "updateLines: ${lines.size} lines")
-        adapter.setContent(lines)
-    }
-
-    private fun alert(message: String, title: String? = null) {
-        val builder = AlertDialog.Builder(this)
-        if (title != null)
-            builder.setTitle(title)
-        else
-            builder.setTitle(title ?: R.string.alert_title)
-        builder
             .setMessage(message)
             .create()
             .show()
     }
 
-    private fun openUnknownScheme(uri: Uri) {
-        try {
-            startActivity(Intent(ACTION_VIEW, uri))
-        } catch (e: ActivityNotFoundException) {
-            alert("Can't open this URL.")
-        }
-    }
-
     class PageViewModel : ViewModel() {
-        private var requestJob: Job? = null
+        private var lines = ArrayList<Line>()
-        val state: MutableLiveData<State> by lazy { MutableLiveData<State>(State.IDLE) }
+        val linesLiveData: MutableLiveData<List<Line>> by lazy { MutableLiveData<List<Line>>() }
-        private var linesList = ArrayList<Line>()
+        val alertLiveData: MutableLiveData<String> by lazy { MutableLiveData<String>() }
-        val lines: MutableLiveData<List<Line>> by lazy { MutableLiveData<List<Line>>() }
-        val alert: MutableLiveData<String> by lazy { MutableLiveData<String>() }
-
-        enum class State {
-            IDLE, CONNECTING, RECEIVING
-        }
 
         /**
          * Perform a request against this URI.
@@ -139,58 +93,43 @@ class MainActivity : AppCompatActivity(), ContentAdapter.ContentAdapterListen {
          * The URI must be valid, absolute and with a gemini scheme.
          */
         fun sendGeminiRequest(uri: Uri) {
-            Log.d(TAG, "sendRequest: URI \"$uri\"")
+            Log.d(TAG, "sendRequest: $uri")
-            state.postValue(State.CONNECTING)
+            viewModelScope.launch(Dispatchers.IO) {
-            requestJob?.apply { if (isActive) cancel() }
-            requestJob = viewModelScope.launch(Dispatchers.IO) {
                 val response = try {
                     val request = Request(uri)
                     val socket = request.connect()
                     val channel = request.proceed(socket, this)
                     Response.from(channel, viewModelScope)
                 } catch (e: UnknownHostException) {
-                    signalError("Unknown host \"${uri.authority}\".")
+                    alertLiveData.postValue("Unknown host \"${uri.authority}\".")
                     return@launch
                 } catch (e: Exception) {
                     Log.e(TAG, "sendGeminiRequest coroutine: ${e.stackTraceToString()}")
-                    signalError("Oops! Whatever we tried to do failed!")
+                    alertLiveData.postValue("Oops! Whatever we tried to do failed!")
                     return@launch
                 }
                 if (response == null) {
-                    signalError("Can't parse server response.")
+                    alertLiveData.postValue("Can't parse server response.")
                     return@launch
                 }
 
                 Log.i(TAG, "sendRequest: got ${response.code} with meta \"${response.meta}\"")
                 when (response.code) {
                     Response.Code.SUCCESS -> handleRequestSuccess(response)
-                    else -> signalError("Can't handle code ${response.code}.")
+                    else -> alertLiveData.postValue("Can't handle code ${response.code}.")
-                }
                 }
             }
 
-        private fun signalError(message: String) {
-            alert.postValue(message)
-            state.postValue(State.IDLE)
         }
 
         private suspend fun handleRequestSuccess(response: Response) {
-            state.postValue(State.RECEIVING)
+            lines.clear()
-            linesList.clear()
-            lines.postValue(linesList)
             val charset = Charset.defaultCharset()
-            var lastUpdate = System.currentTimeMillis()
             for (line in parseData(response.data, charset, viewModelScope)) {
-                linesList.add(line)
+                lines.add(line)
-                val time = System.currentTimeMillis()
+                linesLiveData.postValue(lines)
-                if (time - lastUpdate >= 100) {  // Throttle to 100ms the recycler view updates…
-                    lines.postValue(linesList)
-                    lastUpdate = time
             }
         }
-            lines.postValue(linesList)
-            state.postValue(State.IDLE)
-        }
     }
 
     companion object {

app/src/main/java/dev/lowrespalmtree/comet/Request.kt

@@ -50,7 +50,6 @@ class Request(private val uri: Uri) {
                 }
             }
             Log.d(TAG, "proceed coroutine: reading completed")
-            channel.close()
         }
         return channel
     }

app/src/main/java/dev/lowrespalmtree/comet/Response.kt

@@ -70,7 +70,6 @@ class Response(val code: Code, val meta: String, val data: Channel<ByteArray>) {
                     }
                     // Forward all incoming data to the Response channel.
                     channel.consumeEach { responseChannel.send(it) }
-                    responseChannel.close()
                 }
                 // Return the response here; this stops consuming the channel from this for-loop so
                 // that the coroutine above can take care of it.

app/src/main/res/layout/activity_main.xml

@@ -24,48 +24,28 @@
                 android:layout_width="match_parent"
                 android:layout_height="match_parent"
                 android:layout_margin="4dp"
+                android:ems="10"
                 android:hint="@string/url"
                 android:imeOptions="actionDone|actionGo"
                 android:importantForAutofill="no"
                 android:inputType="textUri"
-                android:text=""
+                android:text="" />
-                tools:ignore="TextContrastCheck" />
 
         </com.google.android.material.appbar.CollapsingToolbarLayout>
 
     </com.google.android.material.appbar.AppBarLayout>
 
-    <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
+    <androidx.core.widget.NestedScrollView
-        android:id="@+id/content_swipe_layout"
+        android:id="@+id/scroll_view"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        app:layout_behavior="@string/appbar_scrolling_view_behavior">
+        app:layout_behavior="com.google.android.material.appbar.AppBarLayout$ScrollingViewBehavior">
-
-        <RelativeLayout
-            android:id="@+id/content_inner_layout"
-            android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
         <androidx.recyclerview.widget.RecyclerView
             android:id="@+id/content_recycler"
             android:layout_width="match_parent"
-                android:layout_height="wrap_content"
+            android:layout_height="match_parent"
-                android:orientation="vertical"
+            android:orientation="vertical" />
-                app:layout_constraintTop_toTopOf="parent" />
+    </androidx.core.widget.NestedScrollView>
-
-            <androidx.core.widget.ContentLoadingProgressBar
-                android:id="@+id/content_progress_bar"
-                style="?android:attr/progressBarStyleHorizontal"
-                android:layout_width="match_parent"
-                android:layout_height="16dp"
-                android:layout_alignParentTop="true"
-                android:layout_marginTop="-8dp"
-                android:indeterminate="true"
-                android:visibility="gone" />
-
-        </RelativeLayout>
-
-    </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>
 
 </androidx.coordinatorlayout.widget.CoordinatorLayout>

app/src/main/res/layout/gemtext_blockquote.xml

@@ -1,12 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="16sp"
-    android:textColor="@color/text"
-    android:textStyle="italic"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="32dp"
+    android:background="@color/teal_200" />
-    android:paddingEnd="16dp"
-    android:paddingTop="4dp"
-    android:paddingBottom="4dp" />

app/src/main/res/layout/gemtext_empty.xml

@@ -1,4 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
-<TextView xmlns:android="http://schemas.android.com/apk/res/android"
+<Space xmlns:android="http://schemas.android.com/apk/res/android"
-    android:layout_width="0dp"
+    android:layout_width="match_parent"
-    android:layout_height="4dp" />
+    android:layout_height="8dp"
+    android:background="@color/black">
+</Space>

app/src/main/res/layout/gemtext_link.xml

@@ -2,11 +2,11 @@
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
     android:textSize="16sp"
-    android:textColor="@color/teal_700"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:paddingStart="8dp"
-    android:paddingEnd="16dp"
+    android:paddingEnd="8dp"
     android:paddingTop="4dp"
     android:paddingBottom="4dp"
+    android:background="@color/teal_200"
     android:textIsSelectable="false" />

app/src/main/res/layout/gemtext_list_item.xml

@@ -1,11 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="16sp"
-    android:textColor="@color/text"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="32dp"
+    android:background="@color/teal_200" />
-    android:paddingEnd="16dp"
-    android:paddingTop="2dp"
-    android:paddingBottom="2dp" />

app/src/main/res/layout/gemtext_paragraph.xml

@@ -2,10 +2,10 @@
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
     android:textSize="16sp"
-    android:textColor="@color/text"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:paddingStart="8dp"
-    android:paddingEnd="16dp"
+    android:paddingEnd="8dp"
     android:paddingTop="4dp"
-    android:paddingBottom="4dp" />
+    android:paddingBottom="4dp"
+    android:background="@color/teal_200" />

app/src/main/res/layout/gemtext_preformatted.xml

@@ -1,13 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="14sp"
-    android:textColor="@color/text"
-    android:typeface="monospace"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:background="@color/teal_200" />
-    android:paddingEnd="8dp"
-    android:paddingTop="2dp"
-    android:paddingBottom="2dp"
-    android:background="@color/purple_200" />

app/src/main/res/layout/gemtext_title1.xml

@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="30sp"
-    android:textColor="@color/text"
-    android:textStyle="bold"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:textSize="30sp"
-    android:paddingEnd="16dp"
+    android:textStyle="bold"
+    android:paddingStart="8dp"
+    android:paddingEnd="8dp"
     android:paddingTop="16dp"
-    android:paddingBottom="4dp" />
+    android:paddingBottom="4dp"
+    android:background="@color/teal_700" />

app/src/main/res/layout/gemtext_title2.xml

@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="24sp"
-    android:textColor="@color/text"
-    android:textStyle="bold"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:textSize="24sp"
-    android:paddingEnd="16dp"
+    android:textStyle="bold"
+    android:paddingStart="8dp"
+    android:paddingEnd="8dp"
     android:paddingTop="8dp"
-    android:paddingBottom="4dp" />
+    android:paddingBottom="4dp"
+    android:background="@color/teal_700" />

app/src/main/res/layout/gemtext_title3.xml

@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="utf-8"?>
 <TextView xmlns:android="http://schemas.android.com/apk/res/android"
     android:id="@+id/text_view"
-    android:textSize="18sp"
-    android:textColor="@color/text"
-    android:textStyle="bold"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingStart="16dp"
+    android:textSize="18sp"
-    android:paddingEnd="16dp"
+    android:textStyle="bold"
+    android:paddingStart="8dp"
+    android:paddingEnd="8dp"
     android:paddingTop="8dp"
-    android:paddingBottom="4dp" />
+    android:paddingBottom="4dp"
+    android:background="@color/teal_700" />

app/src/main/res/values-night/themes.xml

@@ -1,6 +1,6 @@
 <resources xmlns:tools="http://schemas.android.com/tools">
     <!-- Base application theme. -->
-    <style name="Theme.Comet" parent="Theme.MaterialComponents.DayNight.NoActionBar">
+    <style name="Theme.Comet" parent="Theme.MaterialComponents.DayNight.DarkActionBar">
         <!-- Primary brand color. -->
         <item name="colorPrimary">@color/purple_200</item>
         <item name="colorPrimaryVariant">@color/purple_700</item>

app/src/main/res/values/colors.xml

@@ -7,5 +7,4 @@
     <color name="teal_700">#FF018786</color>
     <color name="black">#FF000000</color>
     <color name="white">#FFFFFFFF</color>
-    <color name="text">#222222</color>
 </resources>

app/src/main/res/values/themes.xml

@@ -9,8 +9,6 @@
         <item name="colorSecondary">@color/teal_200</item>
         <item name="colorSecondaryVariant">@color/teal_700</item>
         <item name="colorOnSecondary">@color/black</item>
-        <!-- Status bar color. -->
-        <item name="android:statusBarColor" tools:targetApi="l">?attr/colorPrimaryVariant</item>
         <!-- Customize your theme here. -->
     </style>
 </resources>