rails-markup 1.4.3 → 1.4.4

data/app/assets/javascripts/rails_markup/toolbar/90-init.js

@@ -0,0 +1,91 @@
+  Object.assign(RailsMarkupToolbar, {
+    init(opts = {}) {
+      const previousPathname = this._currentPathname;
+      const previousPageUrl = this._currentPageUrl;
+      const currentPageUrl = this._pageUrl();
+      this.endpoint = opts.endpoint || "/feedback/api";
+      this.accent = opts.accent || "indigo";
+      this.position = opts.position || "bl";
+      this.size = opts.size || "default";
+      this.fabVisible = opts.fabVisible !== false;
+      this.enableScreenshots = opts.enableScreenshots !== false;
+      this.legacyStorageEndpoint = opts.legacyStorageEndpoint || this.legacyStorageEndpoint;
+      this.healthIntervalMs = (opts.healthInterval || 60) * 1000;
+
+      if (document.getElementById("rm-toolbar-root")) {
+        if (currentPageUrl !== this._currentPageUrl) this._onTurboNavigate();
+        return;
+      }
+
+      if (previousPathname && previousPathname !== window.location.pathname) this._deactivateMode();
+      if (previousPageUrl && previousPageUrl !== currentPageUrl && previousPathname === window.location.pathname) this._deactivateMode();
+      this._currentPathname = window.location.pathname;
+      this._currentPageUrl = currentPageUrl;
+      this._injectStyles();
+      this._injectDOM();
+      this._bindEvents();
+      this._loadFromStorage();
+      this._checkHealth();
+      if (!this.healthInterval) this.healthInterval = setInterval(() => this._checkHealth(), this.healthIntervalMs);
+      if (!this._boundVisibilityChange) {
+        this._boundVisibilityChange = () => this._onVisibilityChange();
+        document.addEventListener("visibilitychange", this._boundVisibilityChange);
+      }
+      if (!this._boundOnline) {
+        this._boundOnline = () => this._onOnline();
+        window.addEventListener("online", this._boundOnline);
+      }
+      this._renderPins();
+      this._updateCount();
+      if (previousPageUrl && previousPageUrl !== this._currentPageUrl && this.serverOnline) this._initSession();
+    },
+    destroy() {
+      this._closeAllMenus();
+      this._deactivateMode();
+      if (this.sseSource) { this.sseSource.close(); this.sseSource = null; }
+      if (this.healthInterval) { clearInterval(this.healthInterval); this.healthInterval = null; }
+      if (this._outboxFlushTimer) { clearTimeout(this._outboxFlushTimer); this._outboxFlushTimer = null; }
+      if (this._syncRetryTimer) { clearTimeout(this._syncRetryTimer); this._syncRetryTimer = null; }
+      this._outboxFlushScheduled = false;
+      if (this._boundVisibilityChange) {
+        document.removeEventListener("visibilitychange", this._boundVisibilityChange);
+        this._boundVisibilityChange = null;
+      }
+      if (this._boundOnline) {
+        window.removeEventListener("online", this._boundOnline);
+        this._boundOnline = null;
+      }
+      if (this._boundKeyDown) {
+        document.removeEventListener("keydown", this._boundKeyDown, true);
+        this._boundKeyDown = null;
+      }
+      if (this._onResize) window.removeEventListener("resize", this._onResize);
+      if (this._onScroll) window.removeEventListener("scroll", this._onScroll);
+      if (this._boundTurboFrame) {
+        document.removeEventListener("turbo:frame-render", this._boundTurboFrame);
+        this._boundTurboFrame = null;
+      }
+      if (this._boundMenuDocClick) {
+        document.removeEventListener("click", this._boundMenuDocClick);
+        this._boundMenuDocClick = null;
+      }
+      if (this._boundMenuViewportChange) {
+        document.getElementById("rm-panel-list")
+          ?.removeEventListener("scroll", this._boundMenuViewportChange);
+        window.removeEventListener("resize", this._boundMenuViewportChange);
+        window.removeEventListener("scroll", this._boundMenuViewportChange);
+        this._boundMenuViewportChange = null;
+      }
+      this._onResize = null;
+      this._onScroll = null;
+      const root = document.getElementById("rm-toolbar-root");
+      if (root) root.remove();
+      const pins = document.getElementById("rm-pins-container");
+      if (pins) pins.remove();
+      const styles = document.getElementById("rm-toolbar-styles");
+      if (styles) styles.remove();
+    },
+  });
+
+  global.RailsMarkupToolbar = RailsMarkupToolbar;
+})(typeof window !== "undefined" ? window : this);

data/app/views/rails_markup/shared/_toolbar.html.erb

@@ -7,7 +7,7 @@
     annotations) after a logout Turbo visit whose new body omits the partial. %>
 <span id="rm-toolbar-gate" hidden aria-hidden="true"></span>
 <script>
-<%== File.read(File.expand_path("../../../assets/javascripts/rails_markup/toolbar.js", __dir__)) %>
+<%== RailsMarkup::ToolbarSource.script %>
 </script>
 <script>
   (function() {

data/lib/rails_markup/engine.rb

@@ -7,12 +7,5 @@ module RailsMarkup
     initializer "rails_markup.configuration" do
       RailsMarkup.configuration # ensure defaults are set
     end
-
-    initializer "rails_markup.assets" do |app|
-      if app.config.respond_to?(:assets)
-        # toolbar.js is inlined via the toolbar partial, no separate asset needed
-        app.config.assets.precompile += %w[rails_markup/toolbar.js]
-      end
-    end
   end
 end

data/lib/rails_markup/toolbar_source.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module RailsMarkup
+  module ToolbarSource
+    DIR = File.expand_path("../../app/assets/javascripts/rails_markup/toolbar", __dir__)
+
+    def self.script
+      # Re-read every call in development so editing a toolbar/ module shows up
+      # without a restart (matches the old single-file File.read). Memoize only
+      # where the source can't change under a running process.
+      return build if defined?(Rails) && Rails.env.development?
+
+      @script ||= build
+    end
+
+    def self.build
+      Dir.glob(File.join(DIR, "*.js")).sort.map { |file| File.read(file) }.join("\n")
+    end
+
+    def self.reset!
+      @script = nil
+    end
+  end
+end

data/lib/rails_markup/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsMarkup
-  VERSION = "1.4.3"
+  VERSION = "1.4.4"
 end

data/lib/rails_markup.rb

@@ -2,6 +2,7 @@
 
 require_relative "rails_markup/version"
 require_relative "rails_markup/configuration"
+require_relative "rails_markup/toolbar_source"
 require_relative "rails_markup/store"
 require_relative "rails_markup/http_store_proxy"
 require_relative "rails_markup/http_server"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-markup
 version: !ruby/object:Gem::Version
-  version: 1.4.3
+  version: 1.4.4
 platform: ruby
 authors:
 - InventList
@@ -118,7 +118,14 @@ extra_rdoc_files: []
 files:
 - LICENSE
 - README.md
-- app/assets/javascripts/rails_markup/toolbar.js
+- app/assets/javascripts/rails_markup/toolbar/00-core.js
+- app/assets/javascripts/rails_markup/toolbar/10-styles.js
+- app/assets/javascripts/rails_markup/toolbar/20-dom.js
+- app/assets/javascripts/rails_markup/toolbar/30-menu.js
+- app/assets/javascripts/rails_markup/toolbar/40-events.js
+- app/assets/javascripts/rails_markup/toolbar/50-sync.js
+- app/assets/javascripts/rails_markup/toolbar/60-render.js
+- app/assets/javascripts/rails_markup/toolbar/90-init.js
 - app/controllers/rails_markup/annotations_controller.rb
 - app/controllers/rails_markup/application_controller.rb
 - app/controllers/rails_markup/dashboard_controller.rb
@@ -160,6 +167,7 @@ files:
 - lib/rails_markup/mcp_server.rb
 - lib/rails_markup/server.rb
 - lib/rails_markup/store.rb
+- lib/rails_markup/toolbar_source.rb
 - lib/rails_markup/version.rb
 - lib/tasks/rails_markup_client_uuids.rake
 homepage: https://github.com/nauman/rails-markup