@okjavis/nodebb-theme-javis 1.5.1 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@okjavis/nodebb-theme-javis",
-  "version": "1.5.1",
+  "version": "1.7.0",
   "description": "Modern, premium NodeBB theme for JAVIS Community - Extends Harmony with custom styling",
   "main": "theme.js",
   "scripts": {

package/scss/_base.scss

@@ -30,6 +30,7 @@ nav[component="sidebar/left"] {
   // Sidebar toggle container - push to bottom
   .sidebar-toggle-container {
     margin-top: auto !important; // Push to bottom
+    width: 100% !important;
   }
 }
 
@@ -40,8 +41,8 @@ nav[component="sidebar/left"] {
   padding: 0 20px; // Reddit-style horizontal padding
   justify-content: center; // Center the columns
 
-  // Equal gap between columns (Reddit uses ~16-20px)
-  gap: 20px;
+  // Gap between columns
+  gap: $jv-space-8; // 32px
 
   // Adjust column widths for better proportions
   > [data-widget-area="left"] {

package/scss/_cards.scss

@@ -279,6 +279,20 @@
   gap: $jv-space-3; // 12dp - Material grid
   padding-top: $jv-space-3; // 12dp - Material grid
   margin-top: auto;
+  border-top: none !important; // Remove any border-top
+
+  // Hide hr element if present
+  hr {
+    display: none !important;
+  }
+}
+
+// Hide hr elements within topic cards globally
+.topic-card hr,
+.topic-content hr,
+li[component="post"] hr,
+[component="post"] hr {
+  display: none !important;
 }
 
 .action-btn {

package/scss/_feed.scss

@@ -3,6 +3,24 @@
 // Styles for nodebb-plugin-feed (/feed route)
 // ===========================================================
 
+// ===========================================================
+// HIDE NEW TOPIC BUTTON AND ALL CATEGORIES ROW
+// ===========================================================
+// Hide New Topic button
+// #new_topic,
+// .btn[href*="/compose"],
+// button[data-action="newTopic"],
+// a[href*="/compose"] {
+//   display: none !important;
+// }
+
+// // Hide All categories row
+// .category-selector-container,
+// [component="category/dropdown"],
+// .d-flex.justify-content-between.py-2.mb-2 {
+//   display: none !important;
+// }
+
 // ===========================================================
 // FEED CONTAINER
 // ===========================================================
@@ -10,12 +28,12 @@
   // No max-width - let Bootstrap grid handle width
   // Already inside col-lg-9 which provides proper constraints
 
-  // Header controls row
+  // Header controls row (keeping for reference, but hidden above)
   .d-flex.justify-content-between.py-2.mb-2 {
     padding: $jv-space-4 0 !important;
     margin-bottom: $jv-space-6 !important;
 
-    // New Topic button
+    // New Topic button (hidden above)
     #new_topic,
     .btn-primary {
       font-weight: 600;

package/scss/_sidebar.scss

@@ -51,11 +51,6 @@
   border-right: 1px solid $jv-border-subtle;
   padding-top: 0 !important; // Remove top padding so logo sits at top edge
 
-  // Main navigation list
-  #main-nav {
-    // No gap class - we control spacing per nav-item instead
-  }
-
   // Nav items - proper spacing between items
   .nav-item {
     margin-bottom: $jv-space-2; // 4px between nav items - compact like Slack/Discord
@@ -156,7 +151,6 @@
 div[data-widget-area] {
   display: flex;
   flex-direction: column;
-  gap: $jv-space-6;
 }
 
 // Individual widget card
@@ -178,129 +172,6 @@ div[data-widget-area] {
   }
 }
 
-// Widget titles
-.widget-title,
-div[data-widget-area] h5,
-div[data-widget-area] h6,
-div[data-widget-area="right"] h5,
-div[data-widget-area="right"] h8 {
-  font-size: 14px;
-  font-weight: 700;
-  margin-bottom: $jv-space-6;
-  color: $jv-text-main;
-  text-transform: uppercase;
-  letter-spacing: 0.3px;
-}
-
-// ===========================================================
-// TRENDING TAGS
-// ===========================================================
-
-.popular-tags {
-  gap: 6px;
-
-  .tag-list > div {
-    background: $jv-surface;
-    border: 1px solid $jv-border-subtle;
-    border-radius: 10px;
-    padding: 10px 12px;
-    font-size: 14px;
-    color: $jv-text-main;
-    display: flex;
-    flex-direction: column;
-    gap: 2px;
-    cursor: pointer;
-    transition: border-color $jv-transition-fast, background-color $jv-transition-fast; // Modern transitions
-
-    &:hover {
-      border-color: $jv-border-strong; // Consistent border
-      background: $jv-hover-bg; // Consistent hover state
-    }
-
-    // Focus state for accessibility
-    &:focus-visible {
-      outline: none;
-      box-shadow: $jv-focus-ring;
-    }
-
-    small {
-      color: $jv-text-muted;
-      font-size: $jv-font-size-xs;
-    }
-  }
-}
-
-// ===========================================================
-// TRENDING NOW
-// ===========================================================
-
-#recent_posts {
-  display: flex;
-  flex-direction: column;
-  gap: 10px;
-  margin-top: $jv-space-6;
-
-  li.widget-posts {
-    background: $jv-surface;
-    border: 1px solid rgba(0,0,0,0.05);
-    border-radius: $jv-radius-md;
-    padding: 10px 14px;
-    display: flex;
-    flex-direction: column;
-    gap: $jv-space-4;
-    transition: $jv-transition-shadow, border-color $jv-transition-fast; // Modern transitions
-
-    &:hover {
-      border-color: $jv-border-strong; // Consistent border
-      box-shadow: $jv-shadow-sm; // shadcn subtle shadow
-    }
-
-    // Focus state for accessibility
-    &:focus-within {
-      outline: none;
-      box-shadow: $jv-shadow-md, $jv-focus-ring;
-    }
-
-    // Meta info
-    .d-flex.gap-2,
-    .timeago {
-      font-size: $jv-font-size-xs;
-      color: $jv-text-muted;
-    }
-
-    // Title inside mini post
-    a {
-      font-size: 14px;
-      font-weight: 500;
-      color: $jv-text-main;
-      text-decoration: none;
-      line-height: $jv-line-height-tight;
-
-      &:hover {
-        color: $jv-primary;
-      }
-    }
-
-    // 2-line clamp for content preview
-    p,
-    .content {
-      display: -webkit-box;
-      -webkit-line-clamp: 2;
-      -webkit-box-orient: vertical;
-      overflow: hidden;
-      font-size: $jv-font-size-sm;
-      color: $jv-text-muted;
-      margin: 0;
-    }
-
-    // Read more alignment
-    a.stretched-link {
-      font-size: $jv-font-size-xs;
-      color: $jv-primary;
-    }
-  }
-}
-
 // ===========================================================
 // SEARCH WIDGET
 // ===========================================================
@@ -637,11 +508,35 @@ div[data-widget-area="right"] h8 {
 // ===========================================================
 // RIGHT SIDEBAR WIDGET AREA (Reddit-style)
 // ===========================================================
-div[data-widget-area="sidebar"],
-.col-lg-3[data-widget-area="sidebar"] {
+div[data-widget-area="right"],
+.col-lg-3.col-sm-12 {
   display: flex;
   flex-direction: column;
-  gap: $jv-space-6;
+
+  // Sticky positioning - stays visible when scrolling
+  position: sticky;
+  top: 36px; // Adjust based on your header height
+  align-self: flex-start;
+  max-height: calc(100vh - 100px); // Account for header + padding
+  overflow-y: auto; // Allow scrolling if content is too tall
+
+  // Smooth scrollbar styling (webkit browsers)
+  &::-webkit-scrollbar {
+    width: 6px;
+  }
+
+  &::-webkit-scrollbar-track {
+    background: transparent;
+  }
+
+  &::-webkit-scrollbar-thumb {
+    background: rgba(0, 0, 0, 0.15);
+    border-radius: 3px;
+
+    &:hover {
+      background: rgba(0, 0, 0, 0.25);
+    }
+  }
 
   // Widget card container
   > div {
@@ -651,219 +546,310 @@ div[data-widget-area="sidebar"],
     overflow: hidden;
   }
 
-  // Widget header/title
+  // Widget header/title - clean and simple
   h5, h6, .widget-title {
-    font-size: 12px !important;
+    font-size: 11px !important;
     font-weight: 700 !important;
     text-transform: uppercase;
     letter-spacing: 0.5px;
     color: $jv-text-soft;
-    padding: $jv-space-6 $jv-space-6 $jv-space-4;
+    padding: $jv-space-4 $jv-space-4 $jv-space-4 $jv-space-4;
     margin: 0 !important;
-    border-bottom: 1px solid $jv-border-subtle;
-    background: rgba(0, 0, 0, 0.015);
+    border-bottom: none !important; // Removed border
+  }
+
+  // Add spacing between widgets
+  > div + div {
+    margin-top: $jv-space-4;
   }
 }
 
 // ===========================================================
-// POPULAR TAGS WIDGET (Sidebar)
+// TRENDING TAGS WIDGET (Sidebar) - Pill-style flowing tags
 // ===========================================================
-div[data-widget-area="sidebar"] {
-  .popular-tags,
-  [class*="tag"] {
-    padding: $jv-space-4;
+div[data-widget-area="right"] {
+  // Remove outer card for tags widget - let it flow directly
+  > div:has(.popular-tags) {
+    background: transparent !important;
+    border: none !important;
+    border-radius: 0 !important;
+    overflow: visible !important;
 
-    // Tag items grid
-    .tag-list,
-    > div {
-      display: flex;
-      flex-direction: column;
-      gap: $jv-space-2;
+    // Keep the header styled
+    h5 {
+      padding: $jv-space-3 0 !important;
+      border-bottom: none !important;
+    }
+  }
 
-      > div,
-      > a {
-        display: flex;
-        align-items: center;
-        justify-content: space-between;
-        padding: $jv-space-4 $jv-space-4;
-        background: transparent;
-        border: none;
-        border-radius: $jv-radius-sm;
-        color: $jv-text-main;
-        font-size: 14px;
-        font-weight: 500;
-        text-decoration: none;
-        transition: background-color $jv-transition-fast, color $jv-transition-fast; // Modern transitions
-        cursor: pointer;
+  .popular-tags {
+    padding: 0 !important;
+    margin: 0;
+  }
 
-        &:hover {
-          background: rgba(0, 81, 255, 0.06);
-          color: $jv-primary;
+  // Override Bootstrap's grid - make it a flowing wrap layout
+  .tag-list {
+    display: flex !important;
+    flex-wrap: wrap !important;
+    gap: $jv-space-2 !important; // 8px gap between pills
+    padding: 0 !important;
+    margin: 0 !important;
+
+    // Reset Bootstrap row/column styles
+    &.row {
+      margin: 0 !important;
+      display: flex !important;
+      flex-wrap: wrap !important;
+    }
+
+    // Reset column divs - let them be inline
+    > div {
+      flex: none !important;
+      width: auto !important;
+      padding: 0 !important;
+      margin: 0 !important;
+      max-width: none !important;
+    }
+
+    // Style each tag as a compact pill
+    .btn-ghost,
+    a.btn-ghost {
+      display: inline-flex !important;
+      flex-direction: row !important;
+      align-items: center !important;
+      gap: $jv-space-1 !important; // 4px between tag name and count
+      background: rgba(0, 0, 0, 0.04) !important;
+      border: 1px solid transparent !important;
+      border-radius: $jv-radius-pill !important;
+      // [Enhancement 5] Increased padding for better touch targets
+      padding: $jv-space-2 $jv-space-3 !important; // 8px vertical, 12px horizontal
+      transition: all $jv-transition-fast !important;
+      text-decoration: none !important;
+      width: auto !important;
+      min-height: auto !important;
+
+      &:hover {
+        background: $jv-primary-soft !important;
+        color: $jv-primary !important;
+        // [Enhancement 6] Subtle shadow on hover
+        box-shadow: $jv-shadow-xs !important;
+        transform: translateY(-1px);
+
+        .tag-item {
+          color: $jv-primary !important;
         }
 
-        // Focus state for accessibility
-        &:focus-visible {
-          outline: none;
-          box-shadow: $jv-focus-ring;
+        .tag-topic-count {
+          color: $jv-primary !important;
+          opacity: 0.7;
         }
+      }
+
+      &:focus-visible {
+        outline: none;
+        box-shadow: $jv-focus-ring;
+      }
+
+      .tag-item {
+        font-size: $jv-font-size-xs !important; // 12px
+        font-weight: 500 !important;
+        color: $jv-text-main !important;
+        line-height: 1.2 !important;
+        margin: 0 !important;
 
-        // Topic count
-        small,
-        .count,
-        span:last-child {
-          font-size: 12px;
+        // [Enhancement 2] Add # icon before tag name
+        &::before {
+          content: "#";
           color: $jv-text-soft;
+          margin-right: 2px;
           font-weight: 400;
         }
       }
+
+      .tag-topic-count {
+        font-size: $jv-font-size-xs !important; // 12px
+        font-weight: 400 !important;
+        color: $jv-text-soft !important;
+        line-height: 1 !important;
+
+        // Show count in parentheses style
+        &::before {
+          content: "(";
+        }
+        &::after {
+          content: ")";
+        }
+      }
+
+      // [Enhancement 3] Color-code tags by popularity
+      // Tags with topics get slightly darker background
+      &[data-tag]:not([data-count="0"]) {
+        background: rgba(0, 0, 0, 0.06) !important;
+
+        .tag-item {
+          font-weight: 600 !important;
+        }
+      }
     }
   }
-}
 
-// ===========================================================
-// RECENT/TRENDING POSTS WIDGET (Sidebar)
-// ===========================================================
-div[data-widget-area="sidebar"] {
-  #recent_posts,
-  .recent-posts,
-  ul {
-    list-style: none;
-    padding: 0;
-    margin: 0;
+  // ===========================================================
+  // TRENDING POSTS WIDGET - Widget Essentials Specific
+  // ===========================================================
 
-    li {
-      padding: $jv-space-4 $jv-space-6;
-      border-bottom: 1px solid $jv-border-subtle;
-      transition: background-color $jv-transition-fast; // Modern transition
+  // Override Bootstrap's overflow-hidden on wrapper divs
+  .overflow-hidden {
+    overflow: visible !important;
+  }
 
-      &:last-child {
-        border-bottom: none;
-      }
+  // The wrapper div after h5 (Trending Posts header) - make it scrollable
+  h5 + div {
+    max-height: 420px;
+    overflow-y: auto !important;
+    overflow-x: hidden !important;
 
+    // Scrollbar styling
+    &::-webkit-scrollbar {
+      width: 6px;
+    }
+    &::-webkit-scrollbar-track {
+      background: transparent;
+    }
+    &::-webkit-scrollbar-thumb {
+      background: rgba(0, 0, 0, 0.15);
+      border-radius: 3px;
       &:hover {
-        background: $jv-hover-bg; // Consistent hover state
+        background: rgba(0, 0, 0, 0.25);
       }
+    }
+  }
 
-      // Focus state for accessibility
-      &:focus-within {
-        outline: none;
-        background: $jv-hover-bg;
-      }
+  // Recent posts container - no scroll here, parent wrapper handles it
+  #recent_posts,
+  ul[id*="recent"] {
+    max-height: none !important;
+    overflow: visible !important;
+    padding: $jv-space-3 !important;
+    padding-bottom: $jv-space-5 !important;
+    margin: 0;
+    list-style: none;
+    display: flex !important;
+    flex-direction: column !important;
+    gap: $jv-space-3 !important;
+  }
 
-      // Post header with avatar
-      .d-flex {
-        display: flex;
-        align-items: center;
-        gap: $jv-space-2;
-        margin-bottom: $jv-space-2;
-
-        // Avatar
-        .avatar,
-        img[class*="avatar"] {
-          width: 20px !important;
-          height: 20px !important;
-          border-radius: 50%;
-          flex-shrink: 0;
-        }
+  // Style the actual widget-posts items
+  .widget-posts {
+    padding: $jv-space-3 $jv-space-4 !important;
+    transition: $jv-transition-shadow, border-color $jv-transition-fast !important;
+    border: 1px solid rgba(0,0,0,0.05) !important;
+    border-radius: $jv-radius-md !important;
+    background: $jv-surface !important;
+    cursor: pointer;
+    flex-shrink: 0 !important;
+    display: flex !important;
+    flex-direction: column !important;
+    gap: $jv-space-3 !important;
 
-        // Username
-        a:first-of-type {
-          font-size: 12px;
-          font-weight: 600;
-          color: $jv-text-main;
-          text-decoration: none;
+    &:hover {
+      border-color: $jv-border-strong !important;
+      box-shadow: $jv-shadow-sm;
+    }
 
-          &:hover {
-            color: $jv-primary;
-          }
-        }
+    &:focus-within {
+      outline: none;
+      box-shadow: $jv-shadow-md, $jv-focus-ring;
+    }
 
-        // Timeago
-        .timeago {
-          font-size: 11px;
-          color: $jv-text-soft;
-        }
-      }
+    // Hide hr elements
+    hr {
+      display: none !important;
+    }
 
-      // Post title/content
-      > a,
-      .post-content a,
-      p {
-        display: block;
-        font-size: 13px;
-        font-weight: 500;
-        color: $jv-text-main;
-        text-decoration: none;
-        line-height: 1.4;
-        margin-bottom: $jv-space-2;
+    // Meta info row (author + timeago)
+    .d-flex.gap-2 {
+      display: flex !important;
+      align-items: center !important;
+      gap: $jv-space-2 !important;
+      font-size: $jv-font-size-xs;
+      color: $jv-text-muted;
+      flex-wrap: nowrap;
+    }
 
-        // Line clamp
-        display: -webkit-box;
-        -webkit-line-clamp: 2;
-        -webkit-box-orient: vertical;
-        overflow: hidden;
+    .timeago {
+      font-size: $jv-font-size-xs;
+      color: $jv-text-soft;
+    }
 
-        &:hover {
-          color: $jv-primary;
-        }
-      }
+    .post-author a {
+      font-size: $jv-font-size-xs !important;
+      font-weight: 500 !important;
+      color: $jv-text-main !important;
+    }
 
-      // Content preview text
-      .post-content,
-      .content,
-      .text-xs {
-        font-size: 12px;
-        color: $jv-text-muted;
-        line-height: 1.45;
-        display: -webkit-box;
-        -webkit-line-clamp: 2;
-        -webkit-box-orient: vertical;
-        overflow: hidden;
-      }
+    // Title link
+    a {
+      font-size: 14px;
+      font-weight: 500;
+      color: $jv-text-main;
+      text-decoration: none;
+      line-height: $jv-line-height-tight;
 
-      // Read more link
-      a.stretched-link,
-      .read-more {
-        font-size: 12px;
-        font-weight: 600;
+      &:hover {
         color: $jv-primary;
-        text-decoration: none;
-        margin-top: $jv-space-2;
-        display: inline-block;
+      }
+    }
+
+    // Content preview
+    p,
+    .content,
+    .line-clamp-6 {
+      display: -webkit-box;
+      -webkit-line-clamp: 3;
+      -webkit-box-orient: vertical;
+      overflow: hidden;
+      font-size: $jv-font-size-sm !important;
+      color: $jv-text-muted !important;
+      line-height: $jv-line-height-base !important;
+      margin: 0;
+    }
+
+    // Read more link
+    a.stretched-link {
+      font-size: $jv-font-size-xs;
+      color: $jv-primary;
+    }
+
+    // Footer with "read more" link
+    .post-preview-footer {
+      margin-top: auto;
+
+      a {
+        font-size: $jv-font-size-xs !important;
+        font-weight: 600 !important;
+        color: $jv-primary !important;
+        text-decoration: none !important;
+        transition: color $jv-transition-fast !important;
+        display: inline-flex;
+        align-items: center;
+        gap: $jv-space-1;
 
         &:hover {
-          text-decoration: underline;
+          color: $jv-primary-hover !important;
+          text-decoration: underline !important;
         }
-      }
-    }
-  }
-}
 
-// ===========================================================
-// WIDGET HEADER ICONS
-// ===========================================================
-div[data-widget-area="sidebar"] {
-  h5, h6, .widget-title {
-    display: flex;
-    align-items: center;
-    gap: $jv-space-2;
+        &::after {
+          content: "→";
+          font-size: inherit;
+          transition: transform $jv-transition-fast;
+        }
 
-    i {
-      font-size: 14px;
-      color: $jv-text-soft;
+        &:hover::after {
+          transform: translateX(2px);
+        }
+      }
     }
   }
 }
 
-// ===========================================================
-// EMPTY STATE
-// ===========================================================
-div[data-widget-area="sidebar"] {
-  .empty,
-  .no-content {
-    padding: $jv-space-8;
-    text-align: center;
-    color: $jv-text-soft;
-    font-size: $jv-font-size-sm;
-  }
-}