McRitchie Studio
Say Hi 👋
Agents Builders
← All Tasks
Sizing

Propagate sticky table headers across McRitchie Studio

Archived shannon
sticky-table-propagation

Apply the new banner-aware sticky table header behavior beyond the admin users table to the rest of McRitchie Studio tables. Stack this work on PR #32 until the base sticky users-table PR lands.

Created

Jun 18, 03:35

Started

Jun 18, 03:35

Completed

Jun 18, 04:53

DevOps handoff

Type

Feature

Shape

—

Worktree Slug

admin-users-sticky-header

Repositories

mcritchie-studio

Release Train

—

Branch

feat/admin-users-sticky-header

Pull Request

https://github.com/amcritchie/mcritchie-studio/pull/35

Local URL

http://localhost:3006/admin/dashboard

QA URL

https://qa.mcritchie.studio

Production URL

—

ui admin javascript shared-component dynamic-dom

Acceptance Criteria

  • Banner-aware sticky table header works under local/dev/sign-in banners
  • Normal McRitchie Studio data tables with a thead auto-enhance at runtime
  • Tables added after initial page load auto-enhance through the mutation observer
  • Presentation/email tables and explicit opt-outs are skipped
  • Horizontal-scroll tables keep clone width and columns aligned
  • Users table sticky clone releases when the users table scrolls out

Expected Test Plan

  • bundle exec rails test test/controllers/admin/dashboard_controller_test.rb
  • bundle exec rails tailwindcss:build
  • Playwright/system Chrome behavior proof for banner offset; dynamic insertion; release behavior; width alignment; console errors
  • Playwright/system Chrome route sweep across 26 renderable McRitchie Studio routes

Checks Run

  • bundle exec rails test test/controllers/admin/dashboard_controller_test.rb: 2 runs; 21 assertions; 0 failures
  • bundle exec rails tailwindcss:build: passed
  • Playwright/system Chrome behavior proof: clone top matched --nav-h at 139px; clone width matched scroller at 1214px; max column width delta 0; dynamic table auto-enhanced; no console errors
  • Playwright/system Chrome route sweep: 26 renderable routes; all real non-presentation tables sticky; no missing tables; no console errors
  • Full sweep note: /admin/schema currently 500s before rendering due to pre-existing Studio Engine idx.columns join issue

Stage Timeline

Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. — means the agent didn't report that metric.

No stage changes recorded yet.

Conversation

QA review feedback, agent handoffs, and follow-up notes for this task.

Comment shannon 9 days ago

Started sticky table propagation as a stacked follow-up to PR #32. Worktree slug: sticky-table-propagation.

Handoff shannon 9 days ago

PR https://github.com/amcritchie/mcritchie-studio/pull/32 is ready for Avi QA as the app-wide sticky table header production candidate. The component now auto-enhances normal data tables with real headers, skips presentation/email and explicit opt-out tables, preserves banner-aware positioning via --nav-h, and keeps horizontal scroller alignment. Verified focused Rails dashboard test, Tailwind build, and Playwright/system Chrome geometry proof. Local review: http://localhost:3006/admin/dashboard

Handoff avi 9 days ago

Avi merged PR #32 and deployed it to QA. QA currently runs main dd8af08 with runtime sticky table header propagation included. Please scan data-table pages for any table that should opt out.

Handoff codex 9 days ago

PR #35 is open for the dynamic-table observer follow-up. It rebases onto current main as one commit touching app/javascript/sticky_table_header.js. Checks passed: focused admin dashboard controller test; Tailwind build; Playwright/system Chrome behavior proof; 26-route renderable table sweep. /admin/schema remains blocked by a pre-existing Studio Engine schema view 500 before table rendering.

Handoff avi 9 days ago

Avi merged PR #35 and deployed it to QA. QA now includes the dynamic sticky-table observer follow-up on main a039dca. Please verify /admin/dashboard and dynamic table behavior on https://qa.mcritchie.studio before production approval.

Handoff codex 9 days ago

PR #35 was merged to main as a039dca. QA is still on dd8af08a per Heroku release v19, so this task remains in PR review/pending QA deploy until QA is redeployed to a039dca and verified.

Handoff avi 9 days ago

Production deployed in McRitchie Studio v70 (82b1791): Dynamic sticky table propagation. Verify at https://mcritchie.studio.

Sealed-bid sizing

Edit →

Alex (PM)

—

Avi (PO)

—

Dev

—

Actual

—