Changeset a995698 in Klonkt

Timestamp:

06/28/2026 12:11:39 PM (2 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

0a58300

Parents:

89cc8c4

Message:

fix(ui): hide Fediverse + Notifications nav when AP/federation is off

The fedi nav item + the notifications bell (topnav + mobile profile sheet) were gated only on
canManageFedi, so they stayed visible with federation switched off. Now also gated on apEnabled,
consistent with the rest of the fedi UI which already hides when off. (The profile-sheet
Notifications item was previously ungated entirely.)

Location:

src/views/partials

Files:

• profile-sheet.ejs (modified) (2 diffs)
• topnav.ejs (modified) (2 diffs)

src/views/partials/profile-sheet.ejs

@@ -56 +56 @@
         </a>
       </li>
+      <% if (typeof canManageFedi !== 'undefined' && canManageFedi && apEnabled) { %>
       <li role="none">
         <a href="/notifications" class="profile-sheet-item" role="menuitem" data-close-sheet>
@@ -63 +64 @@
         </a>
       </li>
-      <% if (typeof canManageFedi !== 'undefined' && canManageFedi) { %>
+      <% } %>
+      <% if (typeof canManageFedi !== 'undefined' && canManageFedi && apEnabled) { %>
       <li role="none">
         <a href="/news" class="profile-sheet-item" role="menuitem" data-close-sheet

src/views/partials/topnav.ejs

@@ -101 +101 @@
       <% } %>
 
-      <% if (user && typeof canManageFedi !== 'undefined' && canManageFedi) { %>
+      <% if (user && typeof canManageFedi !== 'undefined' && canManageFedi && apEnabled) { %>
         <a class="nav-btn nav-notif" href="/notifications" aria-label="<%= t('notif.title') %>" title="<%= t('notif.title') %>"
            hx-get="/notifications?partial=1" hx-target="#pcms-main" hx-swap="innerHTML" hx-push-url="/notifications" hx-indicator="#pcms-loading">
@@ -157 +157 @@
                 </a>
               <% } %>
-              <% if (typeof canManageFedi !== 'undefined' && canManageFedi) { %>
+              <% if (typeof canManageFedi !== 'undefined' && canManageFedi && apEnabled) { %>
                 <a href="/news" role="menuitem" class="udi"
                    hx-get="/news?partial=1" hx-target="#pcms-main" hx-swap="innerHTML"