removed rate limit, fixed translations, hidden message on changelog pages

Author: amaksimovv

docs/js/feedback.js

@@ -1,4 +1,19 @@
 (function () {
+  const STRINGS = {
+    ru: {
+      done: "Спасибо!",
+      error: "Не удалось отправить. Попробуйте позже.",
+      missingEndpoint: "Не задан endpoint для фидбэка.",
+      emptyComment: "Добавьте комментарий."
+    },
+    en: {
+      done: "Thanks!",
+      error: "Failed to send. Try later.",
+      missingEndpoint: "Feedback endpoint is not configured.",
+      emptyComment: "Please add a comment."
+    }
+  };
+
   function setText(el, lang) {
     const value = el.getAttribute(lang === "en" ? "data-en" : "data-ru");
     if (value) el.textContent = value;
@@ -14,11 +29,24 @@
     });
   }
 
+  function buildPayload(pageTitle, pageUrl, lang, rating, comment) {
+    return {
+      rating: rating,
+      comment: comment,
+      page_title: pageTitle,
+      page_url: pageUrl,
+      lang: lang,
+      user_agent: navigator.userAgent,
+      created_at: new Date().toISOString()
+    };
+  }
+
   document.addEventListener("DOMContentLoaded", function () {
     const widget = document.querySelector(".cs-feedback");
     if (!widget) return;
 
     const lang = widget.getAttribute("data-lang") || "ru";
+    const strings = STRINGS[lang] || STRINGS.ru;
     const endpoint = widget.getAttribute("data-endpoint") || "";
     const pageTitle = widget.getAttribute("data-page-title") || "";
     const pageUrl = widget.getAttribute("data-page-url") || "";
@@ -27,35 +55,16 @@
     widget.querySelectorAll(".cs-feedback__btn").forEach((btn) => setText(btn, lang));
     widget.querySelectorAll(".cs-feedback__submit").forEach((btn) => setText(btn, lang));
 
-    const status = widget.querySelector(".cs-feedback__status");
     const statusText = widget.querySelector(".cs-feedback__status-text");
     const details = widget.querySelector(".cs-feedback__details");
     const comment = widget.querySelector(".cs-feedback__comment");
     const submit = widget.querySelector(".cs-feedback__submit");
     const trap = widget.querySelector(".cs-feedback__trap");
-    const storageKey = `cs-feedback:${pageUrl || location.pathname}`;
-    const cooldownMs = 2 * 60 * 1000;
 
     function setStatus(text) {
       if (statusText) statusText.textContent = text || "";
     }
 
-    function doneMessage() {
-      return lang === "en" ? "Thanks!" : "Спасибо!";
-    }
-
-    function errorMessage() {
-      return lang === "en" ? "Failed to send. Try later." : "Не удалось отправить. Попробуйте позже.";
-    }
-
-    function requireEndpoint() {
-      return lang === "en" ? "Feedback endpoint is not configured." : "Не задан endpoint для фидбэка.";
-    }
-
-    function alreadySentMessage() {
-      return lang === "en" ? "We already got your feedback. Спасибо!" : "Мы уже получили ваш отзыв. Спасибо!";
-    }
-
     function markDone() {
       widget.classList.add("cs-feedback--done");
     }
@@ -64,20 +73,21 @@
       widget.classList.toggle("cs-feedback--loading", state);
     }
 
-    function saveSent() {
+    async function sendFeedback(rating, value) {
       try {
-        localStorage.setItem(storageKey, String(Date.now()));
-      } catch (e) {
-        // ignore storage errors
-      }
-    }
+        setLoading(true);
+        await postFeedback(endpoint, buildPayload(pageTitle, pageUrl, lang, rating, value));
+        setStatus(strings.done);
+        markDone();
 
-    function isRateLimited() {
-      try {
-        const last = Number(localStorage.getItem(storageKey) || 0);
-        return last && Date.now() - last < cooldownMs;
-      } catch (e) {
-        return false;
+        if (rating === "no") {
+          details.hidden = true;
+          comment.value = "";
+        }
+      } catch (err) {
+        setStatus(strings.error);
+      } finally {
+        setLoading(false);
       }
     }
 
@@ -87,19 +97,13 @@
         setStatus("");
 
         if (trap && trap.value) {
-          setStatus(doneMessage());
-          markDone();
-          return;
-        }
-
-        if (isRateLimited()) {
-          setStatus(alreadySentMessage());
+          setStatus(strings.done);
           markDone();
           return;
         }
 
         if (!endpoint) {
-          setStatus(requireEndpoint());
+          setStatus(strings.missingEndpoint);
           return;
         }
 
@@ -109,73 +113,29 @@
           return;
         }
 
-        try {
-          setLoading(true);
-          await postFeedback(endpoint, {
-            rating: "yes",
-            comment: "",
-            page_title: pageTitle,
-            page_url: pageUrl,
-            lang,
-            user_agent: navigator.userAgent,
-            created_at: new Date().toISOString()
-          });
-          saveSent();
-          setStatus(doneMessage());
-          markDone();
-        } catch (err) {
-          setStatus(errorMessage());
-        } finally {
-          setLoading(false);
-        }
+        await sendFeedback("yes", "");
       });
     });
 
     submit.addEventListener("click", async function () {
       if (trap && trap.value) {
-        setStatus(doneMessage());
-        markDone();
-        return;
-      }
-
-      if (isRateLimited()) {
-        setStatus(alreadySentMessage());
+        setStatus(strings.done);
         markDone();
         return;
       }
 
       if (!endpoint) {
-        setStatus(requireEndpoint());
+        setStatus(strings.missingEndpoint);
         return;
       }
 
       const value = comment.value.trim();
       if (!value) {
-        setStatus(lang === "en" ? "Please add a comment." : "Добавьте комментарий.");
+        setStatus(strings.emptyComment);
         return;
       }
 
-      try {
-        setLoading(true);
-        await postFeedback(endpoint, {
-          rating: "no",
-          comment: value,
-          page_title: pageTitle,
-          page_url: pageUrl,
-          lang,
-          user_agent: navigator.userAgent,
-          created_at: new Date().toISOString()
-        });
-        saveSent();
-        setStatus(doneMessage());
-        markDone();
-        details.hidden = true;
-        comment.value = "";
-      } catch (err) {
-        setStatus(errorMessage());
-      } finally {
-        setLoading(false);
-      }
+      await sendFeedback("no", value);
     });
   });
 })();

docs/stylesheets/extra.css

@@ -461,20 +461,21 @@
 }
 
 .cs-feedback__loader {
+  display: inline-block;
   width: 14px;
   height: 14px;
   border-radius: 50%;
   border: 2px solid rgba(255, 61, 108, 0.35);
   border-top-color: #ff3d6c;
   animation: csFeedbackSpin 0.8s linear infinite;
+  animation-play-state: paused;
   opacity: 0;
-  transform: scale(0.9);
-  transition: opacity 0.2s ease, transform 0.2s ease;
+  transition: opacity 0.2s ease;
 }
 
 .cs-feedback--loading .cs-feedback__loader {
   opacity: 1;
-  transform: scale(1);
+  animation-play-state: running;
 }
 
 @keyframes csFeedbackSpin {

overrides/main.html

@@ -7,8 +7,17 @@
     {% set file_name = page.file.name | default('') | lower %}
     {% set page_url = page.url | default('') %}
     {% set is_index = file_name.startswith('index.') %}
-    {% set is_guide = page_url in ['user-guide/', 'user-guide.en/', 'admin-guide/', 'admin-guide.en/'] %}
-    {% if not page.is_homepage and not is_index and not is_guide %}
+    {% set excluded_feedback_urls = [
+      'user-guide/',
+      'user-guide.en/',
+      'admin-guide/',
+      'admin-guide.en/',
+      'on-premise/how-to/general/',
+      'on-premise/how-to/general.en/'
+    ] %}
+    {% set is_guide = page_url in excluded_feedback_urls %}
+    {% set is_changelog = page_url.startswith('changelog/') %}
+    {% if not page.is_homepage and not is_index and not is_guide and not is_changelog %}
       <div
           class="md-feedback cs-feedback"
           data-lang="{% if '.en/' in page.url or page.url.endswith('.en/') %}en{% else %}ru{% endif %}"