add option to disable showing unreads in the title, also a bit of cleanup

README.md

@@ -23,7 +23,9 @@
 - filters for reactions:
     - matching by regex, domain-specific, or catch-all
     - filters the actual reactions to post as well as notifications
-- option to disable favicon badge when there's unread notifications
+- for unreads:
+    - option to disable favicon badge changing on unreads
+    - option to disable title showing unreads
 
 ---
 

src/components/notifications/notifications.js

@@ -87,7 +87,8 @@ const Notifications = {
       if (count > 0) {
         if (this.$store.getters.mergedConfig.showFaviconBadge)
           FaviconService.drawFaviconBadge()
-        this.$store.dispatch('setPageTitle', `(${count})`)
+        if (this.$store.getters.mergedConfig.showUnreadInTitle)
+          this.$store.dispatch('setPageTitle', `(${count})`)
       } else {
         FaviconService.clearFaviconBadge()
         this.$store.dispatch('setPageTitle', '')

src/components/settings_modal/tabs/general_tab.vue

@@ -709,69 +709,77 @@
     </div>
 
     <div
+      class="setting-item"
       v-if="expertLevel > 0"
     >
-      <div class="setting-item">
-        <h2>{{ $t('settings.requires_refresh') }}</h2>
-        <small>{{ $t('settings.requires_refresh_exp') }}</small>
-        <ul class="setting-list">
-          <li>
-            <BooleanSetting
-              path="autoRefreshOnRequired"
-              expert="1"
-            >
-              {{ $t('settings.auto_refresh_on_required') }}
-            </BooleanSetting>
-          </li>
-          <li>
-            <BooleanSetting
-              path="dedupeBoosts"
-              expert="1"
-              @click="checkAutoReload"
-            >
-              {{ $t('settings.dedupe_boosts') }}
-            </BooleanSetting>
-          </li>
-          <li>
-            <BooleanSetting
-              path="showFaviconBadge"
-              expert="1"
-              @click="checkAutoReload"
-            >
-              {{ $t('settings.show_favicon_badge') }}
-            </BooleanSetting>
-          </li>
-        </ul>
-      </div>
+      <h2>{{ $t('settings.requires_refresh') }}</h2>
+      <small>{{ $t('settings.requires_refresh_exp') }}</small>
+      <ul class="setting-list">
+        <li>
+          <BooleanSetting
+            path="autoRefreshOnRequired"
+            expert="1"
+          >
+            {{ $t('settings.auto_refresh_on_required') }}
+          </BooleanSetting>
+        </li>
+        <li>
+          <BooleanSetting
+            path="dedupeBoosts"
+            expert="1"
+            @click="checkAutoReload"
+          >
+            {{ $t('settings.dedupe_boosts') }}
+          </BooleanSetting>
+        </li>
+        <li>
+          <BooleanSetting
+            path="showFaviconBadge"
+            expert="1"
+            @click="checkAutoReload"
+          >
+            {{ $t('settings.show_favicon_badge') }}
+          </BooleanSetting>
+        </li>
+        <li>
+          <BooleanSetting
+            path="showUnreadInTitle"
+            expert="1"
+            @click="checkAutoReload"
+          >
+            {{ $t('settings.show_unread_in_title') }}
+          </BooleanSetting>
+        </li>
+      </ul>
     </div>
+
     <div
+      class="setting-item"
       v-if="this.$store.getters.mergedConfig.sillyFeatures"
     >
-      <div class="setting-item">
-        <h2>{{ $t('settings.silly_features') }}</h2>
-        <ul class="setting-list">
-          <li>
-            <BooleanSetting
-              path="recurseSearch"
-            >
-              {{ $t('settings.recurse_search') }}
-            </BooleanSetting>
-            <ul
-            class="setting-list suboptions"
-            :class="[{disabled: !recurseSearch}]"
-            >
-              <li>
-                <IntegerSetting
-                  path="recurseSearchLimit"
-                  :min="100"
-                >
-                  {{ $t('settings.recurse_search_limit') }}
-                </IntegerSetting>
-              </li>
-            </ul>
-          </li>
-        </ul>
-      </div>
+      <h2>{{ $t('settings.silly_features') }}</h2>
+      <ul class="setting-list">
+        <li>
+          <BooleanSetting
+            path="recurseSearch"
+          >
+            {{ $t('settings.recurse_search') }}
+          </BooleanSetting>
+          <ul
+          class="setting-list suboptions"
+          :class="[{disabled: !recurseSearch}]"
+          >
+            <li>
+              <IntegerSetting
+                path="recurseSearchLimit"
+                :min="100"
+              >
+                {{ $t('settings.recurse_search_limit') }}
+              </IntegerSetting>
+            </li>
+          </ul>
+        </li>
+      </ul>
     </div>
   </div>
 </template>

src/i18n/en.json

@@ -784,6 +784,7 @@
         "show_panel_nav_shortcuts": "Show timeline navigation shortcuts at the top of the panel",
         "show_scrollbars": "Show side column's scrollbars",
         "show_page_backgrounds": "Show page-specific backgrounds, e.g. for user profiles",
+        "show_unread_in_title": "Show (x) in title for unread notifications",
         "show_wider_shortcuts": "Show wider gap between top panel shortcuts",
         "show_yous": "Show (You)s",
         "silly_features": "Silly features",

src/modules/config.js

@@ -130,6 +130,7 @@ export const defaultState = {
   dedupeBoosts: true,
   showFaviconBadge: true,
   autoRefreshOnRequired: true,
+  showUnreadInTitle: true,
   renderMisskeyMarkdown: undefined,
   renderMfmOnHover: undefined, // instance default
   conversationDisplay: undefined, // instance default

src/modules/instance.js

@@ -86,6 +86,7 @@ const defaultState = {
   dedupeBoosts: true,
   showFaviconBadge: true,
   autoRefreshOnRequired: true,
+  showUnreadInTitle: true,
   renderMisskeyMarkdown: true,
   renderMfmOnHover: false,
   conversationDisplay: 'linear',