src/lib/account.js

@@ -41,7 +41,7 @@ export function parseAccount(data) {
     else
         account.host = get(server).host;
 
-    account.fqn = data.fqn || account.username + "@" + account.host;
+    account.fqn = data.fqn || account.username + account.host;
 
     account.mention = "@" + account.username;
     if (account.host != get(server).host)

src/lib/api.js

@@ -285,17 +285,13 @@ export async function getTimeline(host, token, timeline, max_id, local_only, rem
         headers: { "Authorization": token ? `Bearer ${token}` : null }
     })
 
-    let res = {
-        data: await data.json()
-    }
-
-    if(data.headers.has("Link")) {
     let links = _parseLinkHeader(data.headers.get("Link"));
-        res["prev"] = links.find(f=>f.rel=="prev");
-        res["next"] = links.find(f=>f.rel=="next");
-    }
 
-    return res;
+    return {
+        data: await data.json(),
+        prev: links.find(f=>f.rel=="prev"),
+        next: links.find(f=>f.rel=="next")
+    };
 }
 
 /**
@@ -317,17 +313,13 @@ export async function getFavourites(host, token, max_id) {
         headers: { "Authorization": token ? `Bearer ${token}` : null }
     })
 
-    let res = {
-        data: await data.json()
-    }
-
-    if(data.headers.has("Link")) {
     let links = _parseLinkHeader(data.headers.get("Link"));
-        res["prev"] = links.find(f=>f.rel=="prev");
-        res["next"] = links.find(f=>f.rel=="next");
-    }
 
-    return res;
+    return {
+        data: await data.json(),
+        prev: links.find(f=>f.rel=="prev"),
+        next: links.find(f=>f.rel=="next")
+    };
 }
 
 /**

src/lib/timeline.js

@@ -10,15 +10,12 @@ export const timeline = writable([]);
 const lang = Lang();
 
 let loading = false;
-let last_post = false;      // last post marker, used for fetching next sequence of posts
+let last_post = false;
-let at_end = false;         // at end of timeline, no next param to paginate
 
 export async function getTimeline(timelineType = "home", clean, localOnly = false, remoteOnly = false) {
     if (loading) return; // no spamming!!
     loading = true;
 
-    if(at_end) return;
-
     if(clean) {
         timeline.set([]);
         last_post = false;
@@ -52,11 +49,8 @@ export async function getTimeline(timelineType = "home", clean, localOnly = fals
         return;
     }
 
-    if (!clean && timeline_data.next) {
+    if (!clean) {
         last_post = timeline_data.next.url.searchParams.get("max_id")
-    } else if(!timeline_data.next) {
-        console.log(timeline_data)
-        at_end = true;
     } 
 
     for (let i in timeline_data.data) {
@@ -77,9 +71,3 @@ export async function getTimeline(timelineType = "home", clean, localOnly = fals
     }
     loading = false;
 }
-
-export function clearTimeline() {
-    timeline.set([]);
-    last_post = false;
-    at_end = false;
-}

src/lib/ui/timeline/Timeline.svelte

@@ -1,78 +0,0 @@
-<script>
-    import { page } from '$app/stores';
-    import { account } from '$lib/stores/account.js';
-    import { timeline, getTimeline, clearTimeline } from '$lib/timeline.js';
-    import { app_name } from '$lib/config.js';
-    import Post from '$lib/ui/post/Post.svelte';
-    
-    import Lang from '$lib/lang';
-    const lang = Lang();
-
-    // TODO: refactor to enum when moving to TS
-    export let timelineType;
-
-    $: {
-        // awful hack to update timeline fresh
-        // when timelineType is updated
-        //
-        // TODO: migrate to $effect when migrating to svelte 5
-        timelineType = timelineType
-
-        clearTimeline()
-        getCurrentTimeline()
-    }
-
-    function getCurrentTimeline(clean = false) {
-        switch(timelineType) {
-            case "home":
-                getTimeline("home", clean);
-                break;
-
-            case "local":
-                getTimeline("public", clean, true)
-                break;
-
-            case "federated":
-                getTimeline("public", clean, false, true)
-                break;
-        }
-    }
-
-    account.subscribe(account => {
-        if (account) getCurrentTimeline();
-    });
-
-    document.addEventListener('scroll', () => {
-        if ($account && $page.url.pathname !== "/") return;
-        if (window.innerHeight + window.scrollY >= document.body.offsetHeight - 2048) {
-            getCurrentTimeline();
-        }
-    });
-</script>
-<div id="feed" role="feed">
-    {#if $timeline.length <= 0}
-        <div class="loading throb">
-            <span>{lang.string('timeline.fetching')}</span>
-        </div>
-    {/if}
-    {#each $timeline as post}
-        <Post post_data={post} />
-    {/each}
-</div>
-
-
-<style>
-    #feed {
-        margin-bottom: 20vh;
-    }
-
-    .loading {
-        width: 100%;
-        height: 80vh;
-        display: flex;
-        justify-content: center;
-        align-items: center;
-        font-size: 2em;
-        font-weight: bold;
-    }
-</style>

src/routes/+page.svelte

@@ -9,7 +9,6 @@
     import Button from '$lib/ui/Button.svelte';
     import Post from '$lib/ui/post/Post.svelte';
     import PageHeader from '../lib/ui/core/PageHeader.svelte';
-    import Timeline from '../lib/ui/timeline/Timeline.svelte';
 
     const lang = Lang();
 
@@ -25,7 +24,40 @@
 
         // set in localStorage
         localStorage.setItem(app_name + '_selected_timeline', timelineType);
+     
+        // erase the timeline here so the ui reacts instantly
+        // mae: i could write an awesome undertale reference here
+        timeline.set([]);
+
+        getCurrentTimeline()
     }
+
+    function getCurrentTimeline(clean = false) {
+        switch(timelineType) {
+            case "home":
+                getTimeline("home", clean);
+                break;
+
+            case "local":
+                getTimeline("public", clean, true)
+                break;
+
+            case "federated":
+                getTimeline("public", clean, false, true)
+                break;
+        }
+    }
+
+    account.subscribe(account => {
+        if (account) getCurrentTimeline();
+    });
+
+    document.addEventListener('scroll', () => {
+        if ($account && $page.url.pathname !== "/") return;
+        if (window.innerHeight + window.scrollY >= document.body.offsetHeight - 2048) {
+            getCurrentTimeline();
+        }
+    });
 </script>
 
 {#if $account}
@@ -47,12 +79,24 @@
         </Button>
     </PageHeader>
 
-    <Timeline timelineType={timelineType}/>
+    <div id="feed" role="feed">
+        {#if $timeline.length <= 0}
+            <div class="loading throb">
+                <span>{lang.string('timeline.fetching')}</span>
+            </div>
+        {/if}
+        {#each $timeline as post}
+            <Post post_data={post} />
+        {/each}
+    </div>
 {:else}
     <LoginForm />
 {/if}
 
 <style>
+    #feed {
+        margin-bottom: 20vh;
+    }
 
     .loading {
         width: 100%;