From 2e8fd9a22e9cfd52795df21ffdb1603ae31c05d6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lera=20Elvo=C3=A9?= <yagich@poto.cafe>
Date: Mon, 4 Aug 2025 02:57:51 +0300
Subject: [PATCH] new inbox view

---
 app/__init__.py                  |   8 +--
 app/templates/common/macros.html |   2 +-
 app/templates/mod/user-list.html |   4 +-
 app/templates/users/inbox.html   | 102 ++++++++++++++++++-------------
 data/static/js/ui.js             |   2 +-
 data/static/style.css            |   8 +--
 sass/style.scss                  |   8 +--
 7 files changed, 74 insertions(+), 60 deletions(-)

diff --git a/app/__init__.py b/app/__init__.py
index 1057711..af80e96 100644
--- a/app/__init__.py
+++ b/app/__init__.py
@@ -117,11 +117,11 @@ def create_app():
         return datetime.utcfromtimestamp(ts or int(time.time())).strftime(format)
 
     @app.template_filter("pluralize")
-    def pluralize(number, singular = "", plural = "s"):
-        if number == 1:
-            return singular
+    def pluralize(subject, num=1, singular = "", plural = "s"):
+        if int(num) == 1:
+            return subject + singular
 
-        return plural
+        return subject + plural
 
     @app.template_filter("permission_string")
     def permission_string(term):
diff --git a/app/templates/common/macros.html b/app/templates/common/macros.html
index 960f2eb..0b5cf97 100644
--- a/app/templates/common/macros.html
+++ b/app/templates/common/macros.html
@@ -178,7 +178,7 @@
 {% endif %}
 <div class="accordion {{ "hidden" if hidden else ""}}" style="{{style}}">
   <div class="accordion-header">
-    <button type="button" class="accordion-toggle" {{"disabled" if disabled else ""}}>{{ "►" if hidden else "▼" }}</button>
+    <button type="button" class="accordion-toggle" {{"disabled" if disabled else ""}}>{{ "+" if hidden else "-" }}</button>
     {{ caller('header') }}
   </div>
   <div class="accordion-content {{ "hidden" if hidden else "" }}">
diff --git a/app/templates/mod/user-list.html b/app/templates/mod/user-list.html
index 79ae6ec..3cd660b 100644
--- a/app/templates/mod/user-list.html
+++ b/app/templates/mod/user-list.html
@@ -8,7 +8,7 @@
     {% if section == "header" %}
       <span>Unconfirmed guests</span>
     {% elif section == "content" %}
-      <table class="users-table">
+      <table class="colorful-table">
         <thead>
           <th>Username</th>
           <th class="small">Signed up on</th>
@@ -32,7 +32,7 @@
     {% if section == "header" %}
       <span>Other users</span>
     {% elif section == "content" %}
-      <table class="users-table">
+      <table class="colorful-table">
         <thead>
           <th>Username</th>
           <th class="small">Permission</th>
diff --git a/app/templates/users/inbox.html b/app/templates/users/inbox.html
index 3970eaa..4c93b01 100644
--- a/app/templates/users/inbox.html
+++ b/app/templates/users/inbox.html
@@ -1,51 +1,65 @@
-{% from "common/macros.html" import timestamp, full_post %}
+{% from "common/macros.html" import timestamp, full_post, accordion %}
 {% extends "base.html" %}
 {% block title %}inbox{% endblock %}
 {% block content %}
-<div class="inbox-container">
-  {% if all_subscriptions is none %}
-    You have no subscriptions.<br>
-  {% else %}
-    Your subscriptions:
-    <ul>
-      {% for sub in all_subscriptions %}
-        <li>
-          <a href=" {{ url_for("threads.thread", slug=sub.thread_slug) }} ">{{ sub.thread_title }}</a>
-          <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = sub.thread_slug) }}">
-            <input type="hidden" name="subscribe" value="unsubscribe">
-            <input class="warn" type="submit" value="Unsubscribe">
-          </form>
-        </li>
+<div class="darkbg inbox-container">
+  {% set has_subscriptions = all_subscriptions is not none %}
+  {% call(section) accordion(disabled=not has_subscriptions) %}
+    {% if section == "header" %}
+      {% if not has_subscriptions %}
+        (You have no subscriptions)
+      {% else %}
+        Your subscriptions
+      {% endif %}
+    {% elif section == "content" and has_subscriptions %}
+      <table class="colorful-table">
+        <thead>
+          <th>Thread</th>
+          <th class="small">Unsubscribe</th>
+        </thead>
+        {% for sub in all_subscriptions %}
+          <tr>
+            <td>
+              <a href=" {{ url_for("threads.thread", slug=sub.thread_slug) }} ">{{ sub.thread_title }}</a>
+            </td>
+            <td>
+              <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = sub.thread_slug) }}">
+                <input type="hidden" name="subscribe" value="unsubscribe">
+                <input class="warn" type="submit" value="Unsubscribe">
+              </form>
+            </td>
+          </tr>
+        {% endfor %}
+      </table>
+    {% endif %}
+  {% endcall %}
+  {% if has_subscriptions %}
+    {% if not new_posts %}
+      You have no unread posts.
+    {% else %}
+      You have {{ total_unreads_count }} total unread {{("post" | pluralize(num=total_unreads_count))}}:
+      {% for thread in new_posts %}
+        {% call(section) accordion() %}
+          {% if section == "header" %}
+            {% set latest_post_id = thread.posts[-1].id %}
+            {% set unread_posts_text = " (" + (thread.unread_count | string) + (" unread post" | pluralize(num=thread.unread_count)) %}
+            <a class="accordion-title" href="{{ url_for("threads.thread", slug=latest_post_slug, after=latest_post_id, _anchor="post-" + (latest_post_id | string)) }}" title="Jump to latest post">{{thread.thread_title + unread_posts_text}}, latest at {{ timestamp(thread.newest_post_time) }})</a>
+            <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = thread.thread_slug) }}">
+              <input type="hidden" name="subscribe" value="read">
+              <input type="submit" value="Mark thread as Read">
+            </form>
+            <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = thread.thread_slug) }}">
+              <input type="hidden" name="subscribe" value="unsubscribe">
+              <input class="warn" type="submit" value="Unsubscribe">
+            </form>
+          {% elif section == "content" %}
+            {% for post in thread.posts %}
+              {{ full_post(post, no_reply = true) }}
+            {% endfor %}
+          {% endif %}
+        {% endcall %}
       {% endfor %}
-    </ul>
-  {% endif %}
-  {% if not new_posts %}
-    You have no unread posts.
-  {% else %}
-    You have {{ total_unreads_count }} unread post{{(total_unreads_count | int) | pluralize }}:
-    {% for thread in new_posts %}
-      <div class="accordion">
-        <div class="accordion-header">
-          <button type="button" class="accordion-toggle">▼</button>
-          {% set latest_post_id = thread.posts[-1].id %}
-          {% set unread_posts_text = " (" + (thread.unread_count | string) + (" unread post" | pluralize) %}
-          <a class="accordion-title" href="{{ url_for("threads.thread", slug=latest_post_slug, after=latest_post_id, _anchor="post-" + (latest_post_id | string)) }}" title="Jump to latest post">{{thread.thread_title + unread_posts_text}}, latest at {{ timestamp(thread.newest_post_time) }})</a>
-          <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = thread.thread_slug) }}">
-            <input type="hidden" name="subscribe" value="read">
-            <input type="submit" value="Mark thread as Read">
-          </form>
-          <form class="modform" method="post" action="{{ url_for("threads.subscribe", slug = thread.thread_slug) }}">
-            <input type="hidden" name="subscribe" value="unsubscribe">
-            <input class="warn" type="submit" value="Unsubscribe">
-          </form>
-        </div>
-        <div class="accordion-content">
-          {% for post in thread.posts %}
-            {{ full_post(post, no_reply = true) }}
-          {% endfor %}
-        </div>
-      </div>
-    {% endfor %}
+    {% endif %}
   {% endif %}
 </div>
 {% endblock %}
diff --git a/data/static/js/ui.js b/data/static/js/ui.js
index 9c640bc..38f52cb 100644
--- a/data/static/js/ui.js
+++ b/data/static/js/ui.js
@@ -120,7 +120,7 @@ document.addEventListener("DOMContentLoaded", () => {
       e.stopPropagation();
       accordion.classList.toggle("hidden");
       content.classList.toggle("hidden");
-      toggleButton.textContent = content.classList.contains("hidden") ? "►" : "▼"
+      toggleButton.textContent = content.classList.contains("hidden") ? "+" : "-"
     }
     
     toggleButton.addEventListener("click", toggle);
diff --git a/data/static/style.css b/data/static/style.css
index 0afecda..bc0c2fa 100644
--- a/data/static/style.css
+++ b/data/static/style.css
@@ -570,25 +570,25 @@ input[type=text]:focus, input[type=password]:focus, textarea:focus, select:focus
   border-collapse: collapse;
 }
 
-.users-table {
+.colorful-table {
   border-collapse: collapse;
   width: 100%;
   margin: 10px 0;
   overflow: hidden;
 }
 
-.users-table tr th {
+.colorful-table tr th {
   background-color: #beb1ce;
   padding: 5px 0;
 }
 
-.users-table tr td {
+.colorful-table tr td {
   background-color: rgb(177, 206, 204.5);
   padding: 5px 0;
   text-align: center;
 }
 
-.users-table .small {
+.colorful-table .small {
   width: 250px;
 }
 
diff --git a/sass/style.scss b/sass/style.scss
index a24b672..e5aa2fc 100644
--- a/sass/style.scss
+++ b/sass/style.scss
@@ -567,25 +567,25 @@ input[type="text"], input[type="password"], textarea, select {
   border-collapse: collapse;
 }
 
-.users-table {
+.colorful-table {
   border-collapse: collapse;
   width: 100%;
   margin: 10px 0;
   overflow: hidden;
 }
 
-.users-table tr th {
+.colorful-table tr th {
   background-color: $button_color2;
   padding: 5px 0;
 }
 
-.users-table tr td {
+.colorful-table tr td {
   background-color: $button_color;
   padding: 5px 0;
   text-align: center;
 }
 
-.users-table .small {
+.colorful-table .small {
   width: 250px;
 }