studio-engine 0.141.2 → 0.141.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e37b982954e7699c21010d1269c9f8bb50e3f4b900b580216ab2a61907000cbb
-  data.tar.gz: 2288aaaee5f9f64ed85893a1ea3d5c6f2027316fc00e1da2ef1370e597037994
+  metadata.gz: ca078eb9a5dbe6acfd9979b40081b446fc4a74455e21b9b8417a595225a19b42
+  data.tar.gz: 15c40bf152b5e6a6cd0d8459a22e2a32543a86c2c0821573b10e669afc5da63f
 SHA512:
-  metadata.gz: 9ba01f6b9d3ca61b788e4076c04e1e9699fa602214d026267cf82e0839ea4a2b89f93d66e2ec7b532475bbd32fb77481ae6d4680594ea075c6c0c189f19bcc20
-  data.tar.gz: f6f06cecfe7ef957aef4b61ecc586f972b264f9e13e2ca7502154b947fdcbc9a63a3d138b8c3903bace7f4200be6046ec4dcc41b89d863c70e8e438d30d80e38
+  metadata.gz: 35b1aeb4b523008613f191d4782482464ecf8048b75a7175177c944e4e7f2e170da64ab0435470d442aa6d8be08f83a4cca5bfd0f4e30ee804820cec73e65d12
+  data.tar.gz: de8d944eac95a2f399bd328e2513b95145eb208ea8f38600d188ec6b1ad7069240d4fb4c12324b25b650e66d40d9c45225c1d7374b6f0e66266a523b567233fe

data/CHANGELOG.md

@@ -4,6 +4,13 @@ The format is [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). This pro
 
 ## Unreleased
 
+## 0.141.3 — 2026-11-09
+
+### Fixed
+
+- **Engine migrations pass a host's omakase lint.** Hosts copy `db/migrate/*.rb` as `*.studio_engine.rb` and lint the copies as their own; four engine migrations used `["a"]` where rubocop-rails-omakase wants `[ "a" ]`, which turned moms-app's release static check red. The sources are fixed (whitespace only, no behavior change), and `test/lib/migrations_host_lint_test.rb` runs the omakase config over `db/migrate` so a lint-red migration cannot ship. A host that already copied a migration keeps its copy; a host that hand-fixed one keeps its fix.
+- **The sticky table header's copy no longer draws over the pinned page header.** As a table scrolled out, the copy was pushed up by the table's bottom into the page header's band, and at the same z-index (`--z-sticky`) it painted over the title, because it is appended to `<body>` and wins the tie (Turf Monster `/contests`, My Contests). The copy now stays at the pinned stack's bottom edge and gets shorter as the table leaves, with the copied row moving up inside it, and it stacks at `--z-sticky - 2`, under the page header and the stuck section header. `StudioStickyTableHeaders.placement` now also returns `height` and `shift`; `top` is never above `edge`. No host change.
+
 ## 0.141.2 — 2026-11-09
 
 ### Fixed

data/Gemfile

@@ -110,3 +110,14 @@ end
 group :development, :test do
   gem "importmap-rails", ">= 2.0"
 end
+
+# rubocop-rails-omakase lints the engine's MIGRATIONS with the config a host
+# runs. Hosts copy db/migrate/*.rb into their own tree as *.studio_engine.rb, so
+# an engine migration is linted by every host that installs it; moms-app runs
+# this exact omakase config, and a copied migration once turned its release
+# static check red over `["a"]` vs `[ "a" ]`. test/lib/migrations_host_lint_test.rb
+# runs it over db/migrate only. Not a runtime dependency, and not a whole-engine
+# style rule: the rest of the engine keeps its own style.
+group :development, :test do
+  gem "rubocop-rails-omakase", require: false
+end

data/app/assets/stylesheets/studio/sticky_table_header.css

@@ -25,10 +25,14 @@
   display: none;
   overflow: hidden;
   pointer-events: none;
-  /* --z-sticky: it fakes a sticky <thead>, so a side panel or a menu opened over
-     the table still covers it. The old bare 45 sat between the panel and dropdown
-     tiers for no recorded reason. */
-  z-index: var(--z-sticky, 30);
+  /* TWO RUNGS UNDER --z-sticky: under the pinned page header (--z-sticky) and the
+     stuck section header (--z-sticky - 1), and still over --z-raised content. It
+     fakes a sticky <thead>, so a side panel or a menu opened over the table still
+     covers it. At --z-sticky itself it TIED the page header, and a tie goes to the
+     later element: this one is appended to <body>, so it painted over the title.
+     The script also never places it above the stack's bottom edge
+     (sticky_table_header.js, placement), so the rung is the second guard. */
+  z-index: calc(var(--z-sticky, 30) - 2);
 }
 
 .sticky-table-header-clone table {

data/app/javascript/studio/sticky_table_header.js

@@ -57,11 +57,22 @@
   }
 
   // Where the clone goes for one table, as a pure function of what was measured:
-  // shown once the real header has scrolled up to the edge, and pushed up by the
-  // table's own bottom so it leaves with the table.
+  // shown once the real header has scrolled up to the edge, and gone once the
+  // table's own bottom has.
+  //
+  // IT NEVER RISES ABOVE THE EDGE. As the table leaves, the clone used to be
+  // pushed up by the table's bottom (top = tableBottom - cloneHeight), and that
+  // slid it into the pinned page header's band, where it painted over the title
+  // (Turf Monster /contests: CONTEST / STATUS / ENTRY over "Contests"). A glass
+  // header is translucent, so even a clone stacked under it would show through.
+  // Now the shell stays at the edge and gets SHORTER, `height` = what is left of
+  // the table below the edge, and the copied row moves up inside it by `shift`
+  // (zero or negative), so the row still leaves with the table, cut at the edge.
   function placement({ edge, headerTop, tableBottom, cloneHeight }) {
     const active = headerTop <= edge && tableBottom > edge;
-    return { active, top: active ? Math.min(edge, tableBottom - cloneHeight) : null };
+    if (!active) return { active, top: null, height: null, shift: null };
+    const height = Math.min(cloneHeight, tableBottom - edge);
+    return { active, top: edge, height, shift: height - cloneHeight };
   }
 
   window.StudioStickyTableHeaders = { loaded: true, pinnedEdge, placement };
@@ -151,7 +162,7 @@
       const tableRect = this.table.getBoundingClientRect();
       const scrollerRect = this.scroller.getBoundingClientRect();
       const cloneHeight = headerRect.height || this.cloneShell.getBoundingClientRect().height;
-      const { active, top } = placement({
+      const { active, top, height, shift } = placement({
         edge, headerTop: headerRect.top, tableBottom: tableRect.bottom, cloneHeight
       });
 
@@ -169,9 +180,9 @@
       this.cloneShell.style.left = `${scrollerRect.left}px`;
       this.cloneShell.style.top = `${top}px`;
       this.cloneShell.style.width = `${scrollerRect.width}px`;
-      this.cloneShell.style.height = `${cloneHeight}px`;
+      this.cloneShell.style.height = `${height}px`;
       this.cloneTable.style.width = `${tableRect.width}px`;
-      this.cloneTable.style.transform = `translateX(${translateX}px)`;
+      this.cloneTable.style.transform = `translate(${translateX}px, ${shift}px)`;
       this.table.setAttribute("data-sticky-table-active", "true");
     }
   }

data/app/views/studio/_page_header.html.erb

@@ -58,7 +58,9 @@
     IT JOINS THE STACK WITH data-pin="page" AND POSITIONS OFF --pin-page-top, the
     bottom of everything above it. --pin-stack-bottom includes the row's own edge,
     so a row positioned off it would chase itself down the page. Anything below,
-    such as the sticky table header's clone, sits under the row with no edit.
+    such as the sticky table header's clone, sits under the row: the clone starts
+    at the stack's bottom edge, never above it, and stacks two rungs under the
+    row's --z-sticky (studio/sticky_table_header.css).
 
     THE SENTINEL stays at the row's place in the flow, and the row's distance from
     it is what "stuck" means, on every kind of header. It is absolutely positioned,

data/db/migrate/20260620000001_create_studio_links.rb

@@ -21,7 +21,7 @@ class CreateStudioLinks < ActiveRecord::Migration[7.2]
     add_index :studio_links, :kind
     # Covers both "this owner's links" and "this owner's referral" lookups
     # (referral_for) via the leading columns.
-    add_index :studio_links, [:linkable_type, :linkable_id, :kind],
+    add_index :studio_links, [ :linkable_type, :linkable_id, :kind ],
               name: "idx_studio_links_owner_kind"
   end
 end

data/db/migrate/20260623130000_create_studio_enumerals.rb

@@ -23,8 +23,8 @@ class CreateStudioEnumerals < ActiveRecord::Migration[7.2]
       t.timestamps
     end
 
-    add_index :studio_enumerals, [:category, :key], unique: true
-    add_index :studio_enumerals, [:category, :position]
-    add_index :studio_enumerals, [:category, :rank]
+    add_index :studio_enumerals, [ :category, :key ], unique: true
+    add_index :studio_enumerals, [ :category, :position ]
+    add_index :studio_enumerals, [ :category, :rank ]
   end
 end

data/db/migrate/20260901000001_create_studio_knowledge_docs.rb

@@ -6,8 +6,10 @@
 #
 # STYLE NOTE: reference migrations are written omakase-rubocop-compatible
 # (spaces inside array brackets) because the installed copy is linted by the
-# CONSUMER's rubocop, not this gem's — an engine-styled copy fails the
-# consumer's cert untouched (measured on the Industries adoption ship).
+# CONSUMER's rubocop — an engine-styled copy fails the consumer's cert
+# untouched (measured on the Industries adoption ship).
+# test/lib/migrations_host_lint_test.rb runs that omakase config over
+# db/migrate, so a lint-red migration fails here first.
 class CreateStudioKnowledgeDocs < ActiveRecord::Migration[7.2]
   def change
     create_table :studio_knowledge_docs do |t|

data/db/migrate/20260902000001_create_studio_knowledge_expectations.rb

@@ -5,7 +5,8 @@
 #
 # STYLE NOTE: reference migrations are written omakase-rubocop-compatible
 # (spaces inside array brackets) because the installed copy is linted by the
-# CONSUMER's rubocop, not this gem's.
+# CONSUMER's rubocop. test/lib/migrations_host_lint_test.rb runs that omakase
+# config over db/migrate, so a lint-red migration fails here first.
 class CreateStudioKnowledgeExpectations < ActiveRecord::Migration[7.2]
   def change
     create_table :studio_knowledge_expectations do |t|

data/db/migrate/20261108130000_create_studio_connections.rb

@@ -30,6 +30,6 @@ class CreateStudioConnections < ActiveRecord::Migration[7.2]
       t.timestamps
     end
 
-    add_index :studio_connections, [:provider, :account_key], unique: true
+    add_index :studio_connections, [ :provider, :account_key ], unique: true
   end
 end

data/db/migrate/20261108160000_require_slugs_on_engine_tables.rb

@@ -42,7 +42,7 @@ class RequireSlugsOnEngineTables < ActiveRecord::Migration[7.2]
     "studio_geo_settings" => "geo",
     "studio_site_identities" => "site-identity"
   }.freeze
-  TABLES = (["error_logs"] + NAMED.keys).freeze
+  TABLES = ([ "error_logs" ] + NAMED.keys).freeze
 
   def up
     set_lock_timeout
@@ -111,7 +111,7 @@ class RequireSlugsOnEngineTables < ActiveRecord::Migration[7.2]
   end
 
   def unique_slug_index(table)
-    slug_indexes = connection.indexes(table).select { |index| Array(index.columns) == ["slug"] }
+    slug_indexes = connection.indexes(table).select { |index| Array(index.columns) == [ "slug" ] }
     return if slug_indexes.any?(&:unique)
 
     slug_indexes.each { |index| remove_index table, name: index.name }

data/lib/studio/version.rb

@@ -1,3 +1,3 @@
 module Studio
-  VERSION = "0.141.2"
+  VERSION = "0.141.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: studio-engine
 version: !ruby/object:Gem::Version
-  version: 0.141.2
+  version: 0.141.3
 platform: ruby
 authors:
 - Alex McRitchie
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-11-09 00:00:00.000000000 Z
+date: 2026-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails