From c0c202c5ff2474f533651b593190656e27dc5ac7 Mon Sep 17 00:00:00 2001
From: KodeStar <kode@audiosilo.app>
Date: Wed, 8 Jul 2026 17:10:19 +0100
Subject: [PATCH 1/2] Add a configurable default tag for the dashboard

In tags mode the dashboard always opened showing every link. This adds a
"Default tag" setting (Advanced) that pre-selects one tag group on load, so
the dashboard opens filtered to it - the built-in equivalent of the custom
JavaScript workaround people have been sharing.

The setting is a select populated from the user's own tags, following the
same dynamic-option pattern already used for the search provider. When a tag
is chosen its slug is exposed on the tag list and the matching tab is
activated on load; when the setting is empty, behaviour is unchanged and all
links are shown.

Resolves #1556
---
 app/Http/Controllers/ItemController.php       |  1 +
 app/Setting.php                               | 12 +++++
 database/seeders/SettingsSeeder.php           | 15 ++++++
 lang/en/app.php                               |  1 +
 public/js/app.js                              |  8 ++++
 resources/assets/js/app.js                    |  9 ++++
 resources/views/partials/taglist.blade.php    |  2 +-
 tests/Feature/DashTest.php                    | 22 +++++++++
 .../database/seeders/SettingsSeederTest.php   | 47 +++++++++++++++++++
 9 files changed, 116 insertions(+), 1 deletion(-)

diff --git a/app/Http/Controllers/ItemController.php b/app/Http/Controllers/ItemController.php
index cd5492b1e..87a383089 100644
--- a/app/Http/Controllers/ItemController.php
+++ b/app/Http/Controllers/ItemController.php
@@ -40,6 +40,7 @@ public function dash(Request $request): View
         $treat_tags_as = \App\Setting::fetch('treat_tags_as');
 
         $data["treat_tags_as"] = $treat_tags_as;
+        $data['default_tag'] = \App\Setting::fetch('default_tag');
 
         if (config('app.auth_roles_enable')) {
             $roles = explode(config('app.auth_roles_delimiter'), $request->header(config('app.auth_roles_header')));
diff --git a/app/Setting.php b/app/Setting.php
index 0c6dd5fdf..72768d363 100644
--- a/app/Setting.php
+++ b/app/Setting.php
@@ -123,6 +123,12 @@ public function getListValueAttribute()
                     $options = (array) json_decode($this->options);
                     if ($this->key === 'search_provider') {
                         $options = Search::providers()->pluck('name', 'id')->toArray();
+                    } elseif ($this->key === 'default_tag') {
+                        $options = [];
+                        $tags = Item::where('type', 1)->where('id', '>', 0)->orderBy('title', 'asc')->get();
+                        foreach ($tags as $tag) {
+                            $options[$tag->tag_url] = $tag->title;
+                        }
                     }
                     $value = (array_key_exists($this->value, $options))
                         ? __($options[$this->value])
@@ -190,6 +196,12 @@ public function getEditValueAttribute()
                 $options = json_decode($this->options);
                 if ($this->key === 'search_provider') {
                     $options = Search::providers()->pluck('name', 'id');
+                } elseif ($this->key === 'default_tag') {
+                    $options = ['' => 'app.options.none'];
+                    $tags = Item::where('type', 1)->where('id', '>', 0)->orderBy('title', 'asc')->get();
+                    foreach ($tags as $tag) {
+                        $options[$tag->tag_url] = $tag->title;
+                    }
                 }
                 $value = '<select name="value" class="form-control">';
                 foreach ($options as $key => $opt) {
diff --git a/database/seeders/SettingsSeeder.php b/database/seeders/SettingsSeeder.php
index 8618c5263..e47deb3ed 100755
--- a/database/seeders/SettingsSeeder.php
+++ b/database/seeders/SettingsSeeder.php
@@ -349,5 +349,20 @@ public function run(): void
             $setting->label = 'app.settings.treat_tags_as';
             $setting->save();
         }
+
+        if (! $setting = Setting::find(15)) {
+            $setting = new Setting;
+            $setting->id = 15;
+            $setting->group_id = 4;
+            $setting->key = 'default_tag';
+            $setting->type = 'select';
+            $setting->label = 'app.settings.default_tag';
+            $setting->value = '';
+            $setting->save();
+        } else {
+            $setting->group_id = 4;
+            $setting->label = 'app.settings.default_tag';
+            $setting->save();
+        }
     }
 }
diff --git a/lang/en/app.php b/lang/en/app.php
index 3c31f53d6..04c8b307b 100644
--- a/lang/en/app.php
+++ b/lang/en/app.php
@@ -29,6 +29,7 @@
   'settings.custom_css' => 'Custom CSS',
   'settings.custom_js' => 'Custom JavaScript',
   'settings.treat_tags_as' => 'Treat Tags As:',
+  'settings.default_tag' => 'Default tag',
   'settings.folders' => 'Folders',
   'settings.tags' => 'Tags',
   'settings.categories' => 'Categories',
diff --git a/public/js/app.js b/public/js/app.js
index 2e50d801b..455e2b0a6 100644
--- a/public/js/app.js
+++ b/public/js/app.js
@@ -4287,6 +4287,14 @@ $.when($.ready).then(function () {
       alert("Something went wrong: ".concat(responseData.responseText.substring(0, 100)));
     });
   });
+  // Auto-select the configured default tag on load (tags mode only)
+  var taglist = document.getElementById("taglist");
+  if (taglist !== null) {
+    var defaultTag = taglist.getAttribute("data-default-tag");
+    if (typeof defaultTag === "string" && defaultTag !== "") {
+      $("#taglist .tag[data-tag=\"tag-".concat(defaultTag, "\"]")).trigger("click");
+    }
+  }
   $("#pinlist").on("click", "a", function (e) {
     e.preventDefault();
     var current = $(this);
diff --git a/resources/assets/js/app.js b/resources/assets/js/app.js
index 42f31c81b..abe633d91 100644
--- a/resources/assets/js/app.js
+++ b/resources/assets/js/app.js
@@ -334,6 +334,15 @@ $.when($.ready).then(() => {
           );
         });
     });
+  // Auto-select the configured default tag on load (tags mode only)
+  const taglist = document.getElementById("taglist");
+  if (taglist !== null) {
+    const defaultTag = taglist.getAttribute("data-default-tag");
+    if (typeof defaultTag === "string" && defaultTag !== "") {
+      $(`#taglist .tag[data-tag="tag-${defaultTag}"]`).trigger("click");
+    }
+  }
+
   $("#pinlist").on("click", "a", function (e) {
     e.preventDefault();
     const current = $(this);
diff --git a/resources/views/partials/taglist.blade.php b/resources/views/partials/taglist.blade.php
index 741832fc9..e1b9e4da9 100644
--- a/resources/views/partials/taglist.blade.php
+++ b/resources/views/partials/taglist.blade.php
@@ -3,7 +3,7 @@
 ?>
 @if( $treat_tags_as == 'tags')
     @if($taglist->first())
-        <div id="taglist" class="taglist">
+        <div id="taglist" class="taglist" data-default-tag="{{ \App\Setting::fetch('default_tag') }}">
             <div class="tag white current" data-tag="all">All</div>
             @foreach($taglist as $tag)
                 <div class="tag link{{ title_color($tag->colour) }}" style="background-color: {{ $tag->colour }}" data-tag="tag-{{ $tag->tag_url }}">{{ $tag->title }}</div>
diff --git a/tests/Feature/DashTest.php b/tests/Feature/DashTest.php
index 189436b7b..8b52c0129 100644
--- a/tests/Feature/DashTest.php
+++ b/tests/Feature/DashTest.php
@@ -4,6 +4,7 @@
 
 use App\Item;
 use App\ItemTag;
+use App\Setting;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;
 
@@ -80,4 +81,25 @@ public function test_displays_tags_on_the_dash(): void
         $response->assertSee('Tag 1');
         $response->assertSee('Tag 2');
     }
+
+    public function test_dash_exposes_the_configured_default_tag(): void
+    {
+        $this->seed();
+
+        Setting::where('key', 'treat_tags_as')->update(['value' => 'tags']);
+        Setting::where('key', 'default_tag')->update(['value' => 'home-dashboard']);
+
+        Item::factory()->create([
+            'title' => 'Home',
+            'url' => 'home-dashboard',
+            'type' => 1,
+            'pinned' => 1,
+            'user_id' => 0,
+        ]);
+
+        $response = $this->get('/');
+
+        $response->assertStatus(200);
+        $response->assertSee('data-default-tag="home-dashboard"', false);
+    }
 }
diff --git a/tests/Unit/database/seeders/SettingsSeederTest.php b/tests/Unit/database/seeders/SettingsSeederTest.php
index d3e2e0e56..01bf6701f 100644
--- a/tests/Unit/database/seeders/SettingsSeederTest.php
+++ b/tests/Unit/database/seeders/SettingsSeederTest.php
@@ -2,11 +2,16 @@
 
 namespace Tests\Unit\database\seeders;
 
+use App\Item;
+use App\Setting;
 use Database\Seeders\SettingsSeeder;
+use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;
 
 class SettingsSeederTest extends TestCase
 {
+    use RefreshDatabase;
+
     /**
      * All language keys are defined in all languages based on the en language file.
      */
@@ -18,4 +23,46 @@ public function test_returns_a_jsonmap_with_same_amount_of_items_as_language_dir
 
         $this->assertTrue(count($languageMap) === count($languageDirectories));
     }
+
+    public function test_seeds_the_default_tag_setting(): void
+    {
+        $this->seed();
+
+        $setting = Setting::where('key', 'default_tag')->first();
+
+        $this->assertNotNull($setting);
+        $this->assertSame('select', $setting->type);
+        $this->assertSame(4, (int) $setting->group_id);
+    }
+
+    public function test_default_tag_edit_value_lists_all_tags_and_a_none_option(): void
+    {
+        $this->seed();
+
+        Item::factory()->create([
+            'title' => 'Home',
+            'url' => 'home-dashboard',
+            'type' => 1,
+            'user_id' => 0,
+        ]);
+        Item::factory()->create([
+            'title' => 'Media',
+            'url' => 'media',
+            'type' => 1,
+            'user_id' => 0,
+        ]);
+
+        $setting = Setting::where('key', 'default_tag')->first();
+        $editValue = $setting->edit_value;
+
+        // A "none" option with an empty value, using the shared translation key.
+        $this->assertStringContainsString('<option value="" ', $editValue);
+        $this->assertStringContainsString(__('app.options.none'), $editValue);
+
+        // One option per tag: the slug as the value, the raw title as the label.
+        $this->assertStringContainsString('value="home-dashboard"', $editValue);
+        $this->assertStringContainsString('>Home</option>', $editValue);
+        $this->assertStringContainsString('value="media"', $editValue);
+        $this->assertStringContainsString('>Media</option>', $editValue);
+    }
 }

From ad9baffa627172357f2eb84166ba6555166803cf Mon Sep 17 00:00:00 2001
From: KodeStar <kode@audiosilo.app>
Date: Wed, 8 Jul 2026 18:26:19 +0100
Subject: [PATCH 2/2] Only offer pinned tags as the default tag

The default_tag dropdown was populated from every tag (type=1), but the
dashboard taglist only renders pinned tags. Selecting an unpinned tag as the
default therefore triggered a click on a taglist entry that does not exist,
silently doing nothing. Filter the option queries in both Setting accessors to
pinned tags so only selectable tags are offered, and assert an unpinned tag is
excluded.

Also drop the unused $data['default_tag'] assignment in ItemController: the
taglist partial reads the setting directly via Setting::fetch(), so the view
variable was never consumed.
---
 app/Http/Controllers/ItemController.php          |  1 -
 app/Setting.php                                  |  4 ++--
 .../Unit/database/seeders/SettingsSeederTest.php | 16 +++++++++++++++-
 3 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/app/Http/Controllers/ItemController.php b/app/Http/Controllers/ItemController.php
index 87a383089..cd5492b1e 100644
--- a/app/Http/Controllers/ItemController.php
+++ b/app/Http/Controllers/ItemController.php
@@ -40,7 +40,6 @@ public function dash(Request $request): View
         $treat_tags_as = \App\Setting::fetch('treat_tags_as');
 
         $data["treat_tags_as"] = $treat_tags_as;
-        $data['default_tag'] = \App\Setting::fetch('default_tag');
 
         if (config('app.auth_roles_enable')) {
             $roles = explode(config('app.auth_roles_delimiter'), $request->header(config('app.auth_roles_header')));
diff --git a/app/Setting.php b/app/Setting.php
index 72768d363..74532ef70 100644
--- a/app/Setting.php
+++ b/app/Setting.php
@@ -125,7 +125,7 @@ public function getListValueAttribute()
                         $options = Search::providers()->pluck('name', 'id')->toArray();
                     } elseif ($this->key === 'default_tag') {
                         $options = [];
-                        $tags = Item::where('type', 1)->where('id', '>', 0)->orderBy('title', 'asc')->get();
+                        $tags = Item::where('type', 1)->where('id', '>', 0)->pinned()->orderBy('title', 'asc')->get();
                         foreach ($tags as $tag) {
                             $options[$tag->tag_url] = $tag->title;
                         }
@@ -198,7 +198,7 @@ public function getEditValueAttribute()
                     $options = Search::providers()->pluck('name', 'id');
                 } elseif ($this->key === 'default_tag') {
                     $options = ['' => 'app.options.none'];
-                    $tags = Item::where('type', 1)->where('id', '>', 0)->orderBy('title', 'asc')->get();
+                    $tags = Item::where('type', 1)->where('id', '>', 0)->pinned()->orderBy('title', 'asc')->get();
                     foreach ($tags as $tag) {
                         $options[$tag->tag_url] = $tag->title;
                     }
diff --git a/tests/Unit/database/seeders/SettingsSeederTest.php b/tests/Unit/database/seeders/SettingsSeederTest.php
index 01bf6701f..afed4bc65 100644
--- a/tests/Unit/database/seeders/SettingsSeederTest.php
+++ b/tests/Unit/database/seeders/SettingsSeederTest.php
@@ -43,12 +43,23 @@ public function test_default_tag_edit_value_lists_all_tags_and_a_none_option():
             'title' => 'Home',
             'url' => 'home-dashboard',
             'type' => 1,
+            'pinned' => 1,
             'user_id' => 0,
         ]);
         Item::factory()->create([
             'title' => 'Media',
             'url' => 'media',
             'type' => 1,
+            'pinned' => 1,
+            'user_id' => 0,
+        ]);
+        // An unpinned tag is not rendered in the dashboard taglist, so it must
+        // not be offered as a default (selecting it would silently do nothing).
+        Item::factory()->create([
+            'title' => 'Archive',
+            'url' => 'archive',
+            'type' => 1,
+            'pinned' => 0,
             'user_id' => 0,
         ]);
 
@@ -59,10 +70,13 @@ public function test_default_tag_edit_value_lists_all_tags_and_a_none_option():
         $this->assertStringContainsString('<option value="" ', $editValue);
         $this->assertStringContainsString(__('app.options.none'), $editValue);
 
-        // One option per tag: the slug as the value, the raw title as the label.
+        // One option per pinned tag: the slug as the value, the raw title as the label.
         $this->assertStringContainsString('value="home-dashboard"', $editValue);
         $this->assertStringContainsString('>Home</option>', $editValue);
         $this->assertStringContainsString('value="media"', $editValue);
         $this->assertStringContainsString('>Media</option>', $editValue);
+
+        // The unpinned tag is excluded.
+        $this->assertStringNotContainsString('value="archive"', $editValue);
     }
 }