feedback_engine 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 156ebb1c4481df5db12c92701ca07f5199f033aa967f264aa855caf327f4542a
-  data.tar.gz: f59ef8d27b7539d16f03eaef38abc4d6ce51ee55cdd630a738433ca7f3463ef7
+  metadata.gz: 12457e35c33daf8032e5eae4336964217b969d4d45d8d8a13ca85f6706833e26
+  data.tar.gz: 82ce331114ef1a88718e9879f7a263c6ea89456d0fb32c098804431df8fa6106
 SHA512:
-  metadata.gz: 827983bd43f6d4b9af39b6d526f2d14755365ce561cfdbb338e240e97a2a54afa2173e0efb645846078d107c7ee234de27356e9814acbdbd891f8d7116e6a30b
-  data.tar.gz: 3f311aeb99159a9d538ddd87e8d346d7b2bd7d10b4521fcec6e864945801eba2ccf9a5e58bf2ebd65034189379647dabf50d60bc5e421924cb2b012468953481
+  metadata.gz: 16ab901cdc39fbd5dfa1c26ef896717610f8da85a2afd96348bfbaa1b13b196b4d9e12e6fb8d47f2194d0bd86e6ba46729f88e86a3e5089f853ab9dbdfff097c
+  data.tar.gz: ee81a5410c06cfaaea26f143ef0ce2bccde945d2d1826e68e30b90f154191e4178cac1f2b2f8279465ccf22b1f20810ca8f9aa2828f6ab184594e7582b56ec2a

data/CHANGELOG.md

@@ -1,11 +1,17 @@
 # Changelog
 
+## 0.4.0 (2026-07-22)
+
+- Removed the clipboard-paste / drag-and-drop / file-chips screenshot intake
+  added in 0.3.0 — pasted files all arrive as `image.png` and the extra UI
+  wasn't worth its complexity. Screenshots are back to a plain file input.
+
 ## 0.3.0 (2026-07-22)
 
-- Screenshots can now be pasted (Cmd/Ctrl+V) or dragged & dropped anywhere on
+- Screenshots can be pasted (Cmd/Ctrl+V) or dragged & dropped anywhere on
   the feedback form, not just picked via the file dialog. Selected files show
   as removable chips; non-images are ignored and the configured maximum is
-  enforced at intake.
+  enforced at intake. (Removed again in 0.4.0.)
 - The dialog traps Tab focus while open and is labelled for screen readers
   (`aria-labelledby`).
 - Dashboard search across message, author, and section (case-insensitive,

data/README.md

@@ -20,8 +20,7 @@ app.
 - **Trigger it your way.** Use the built-in floating button, or hide it and
   open the form from any element with a `data-feedback-engine-open` attribute.
 - **Screenshots included.** Users can attach up to 3 images (via Active
-  Storage) — picked, pasted (Cmd/Ctrl+V), or dragged onto the form. Limits are
-  configurable and enforced server-side.
+  Storage) — limits are configurable and enforced server-side.
 - **Pluggable gating and attribution.** You decide who can send feedback, who
   can read it, and how a submission is attributed to a user.
 - **Localized.** The widget follows your app's `I18n.locale`; translations ship
@@ -177,11 +176,9 @@ end
 ### Screenshots
 
 Uploads use Active Storage: `bin/rails active_storage:install` if you haven't
-already. Users can pick files, paste an image from the clipboard, or drag &
-drop onto the form; selected files show as removable chips. Limits
-(`max_screenshots`, `max_screenshot_size`) are enforced server-side and shown
-as a hint in the widget. If Active Storage isn't loaded, the upload control
-simply doesn't render and uploads are rejected.
+already. Limits (`max_screenshots`, `max_screenshot_size`) are enforced
+server-side and shown as a hint in the widget. If Active Storage isn't loaded,
+the upload control simply doesn't render and uploads are rejected.
 
 ### Notifications
 

data/app/assets/feedback_engine/widget.js

@@ -20,8 +20,6 @@
   var Z = 2147483000;
   var overlay = null;
   var lastFocused = null;
-  var fileInput = null;
-  var fileChips = null;
 
   function ready(fn) {
     if (document.readyState === "loading") {
@@ -114,25 +112,6 @@
     // Keep Tab (and Shift+Tab) cycling inside the dialog while it is open.
     overlay.addEventListener("keydown", trapFocus);
 
-    // Screenshots can be pasted (Cmd/Ctrl+V) or dropped anywhere on the form —
-    // the file picker is just one way in.
-    if (config.screenshots.enabled) {
-      dialog.addEventListener("paste", function (event) {
-        var files = event.clipboardData && event.clipboardData.files;
-        if (files && files.length) {
-          event.preventDefault();
-          addFiles(files);
-        }
-      });
-      dialog.addEventListener("dragover", function (event) {
-        event.preventDefault();
-      });
-      dialog.addEventListener("drop", function (event) {
-        event.preventDefault();
-        if (event.dataTransfer) addFiles(event.dataTransfer.files);
-      });
-    }
-
     var first = dialog.querySelector("select, textarea, input");
     if (first) first.focus();
   }
@@ -141,8 +120,6 @@
     if (!overlay) return;
     overlay.remove();
     overlay = null;
-    fileInput = null;
-    fileChips = null;
     if (lastFocused && lastFocused.focus) lastFocused.focus();
   }
 
@@ -265,69 +242,19 @@
   }
 
   function screenshotsField() {
-    fileInput = document.createElement("input");
-    fileInput.type = "file";
-    fileInput.name = "screenshots";
-    fileInput.multiple = true;
-    fileInput.accept = "image/*";
-    fileInput.addEventListener("change", renderFileChips);
-    var wrap = field(config.labels.screenshots, fileInput);
+    var input = document.createElement("input");
+    input.type = "file";
+    input.name = "screenshots";
+    input.multiple = true;
+    input.accept = "image/*";
+    var wrap = field(config.labels.screenshots, input);
     var hint = document.createElement("span");
     hint.className = "fbe-hint";
     hint.textContent = config.labels.screenshotsHint;
     wrap.appendChild(hint);
-    fileChips = document.createElement("ul");
-    fileChips.className = "fbe-chips";
-    wrap.appendChild(fileChips);
     return wrap;
   }
 
-  // Merge pasted/dropped images into the file input (the single source of
-  // truth for what gets uploaded), capped at the configured maximum.
-  function addFiles(files) {
-    if (!fileInput) return;
-    var transfer = new DataTransfer();
-    var current = Array.prototype.slice.call(fileInput.files);
-    var incoming = Array.prototype.slice.call(files).filter(function (file) {
-      return /^image\//.test(file.type);
-    });
-    current.concat(incoming).slice(0, config.screenshots.max).forEach(function (file) {
-      transfer.items.add(file);
-    });
-    fileInput.files = transfer.files;
-    renderFileChips();
-  }
-
-  function removeFile(index) {
-    if (!fileInput) return;
-    var transfer = new DataTransfer();
-    Array.prototype.slice.call(fileInput.files).forEach(function (file, i) {
-      if (i !== index) transfer.items.add(file);
-    });
-    fileInput.files = transfer.files;
-    renderFileChips();
-  }
-
-  function renderFileChips() {
-    if (!fileChips) return;
-    fileChips.textContent = "";
-    Array.prototype.slice.call(fileInput.files).forEach(function (file, index) {
-      var chip = document.createElement("li");
-      var name = document.createElement("span");
-      name.textContent = file.name;
-      var remove = document.createElement("button");
-      remove.type = "button";
-      remove.setAttribute("aria-label", config.labels.close + " " + file.name);
-      remove.textContent = "×";
-      remove.addEventListener("click", function () {
-        removeFile(index);
-      });
-      chip.appendChild(name);
-      chip.appendChild(remove);
-      fileChips.appendChild(chip);
-    });
-  }
-
   // --- submit -------------------------------------------------------------------
 
   function submit(form) {
@@ -436,12 +363,6 @@
       "border:1px solid #d1d5db;border-radius:6px;background:none;color:#1c2024;font:inherit;cursor:pointer}",
       "#fbe-dialog textarea{resize:vertical}",
       "#fbe-dialog .fbe-hint{display:block;margin-top:4px;font-size:12px;color:#6b7280;font-weight:400}",
-      "#fbe-dialog .fbe-chips{list-style:none;margin:6px 0 0;padding:0;display:flex;flex-wrap:wrap;gap:6px}",
-      "#fbe-dialog .fbe-chips li{display:flex;align-items:center;gap:4px;max-width:100%;",
-      "padding:2px 4px 2px 10px;border:1px solid #d1d5db;border-radius:999px;font-size:12px;font-weight:400}",
-      "#fbe-dialog .fbe-chips span{overflow:hidden;text-overflow:ellipsis;white-space:nowrap;max-width:180px}",
-      "#fbe-dialog .fbe-chips button{border:0;background:none;color:inherit;cursor:pointer;",
-      "font-size:14px;line-height:1;padding:2px 6px}",
       "#fbe-dialog .fbe-error{color:#dc2626;margin:0 0 12px}",
       "#fbe-dialog .fbe-actions{display:flex;justify-content:flex-end;gap:8px}",
       "#fbe-dialog button{padding:8px 14px;border-radius:8px;cursor:pointer;font:inherit}",
@@ -454,7 +375,6 @@
       "#fbe-dialog select,#fbe-dialog textarea,#fbe-dialog input[type=file]{border-color:#2a313a}",
       "#fbe-dialog input[type=file]{color:#9aa2ab}",
       "#fbe-dialog input[type=file]::file-selector-button{border-color:#2a313a;color:#e6e8ea}",
-      "#fbe-dialog .fbe-chips li{border-color:#2a313a}",
       "#fbe-dialog .fbe-secondary{border-color:#2a313a}",
       "#fbe-dialog .fbe-hint{color:#9aa2ab}",
       "}"

data/lib/feedback_engine/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FeedbackEngine
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: feedback_engine
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Yaroslav Shmarov