HOLD: Add page-header design reference (styleguide)#1688
Draft
maebeale wants to merge 1 commit into
Draft
Conversation
maebeale
commented
Jun 15, 2026
| class StyleguideController < ApplicationController | ||
| # Internal design reference for comparing page-header treatments. | ||
| # Logged-in only; no model data required, so authorization is skipped. | ||
| skip_verify_authorized |
Collaborator
Author
There was a problem hiding this comment.
🤖 From Claude: Auth is skipped (not authorize!) because this is a static, data-free internal reference with no records to authorize against. Still behind authenticate_user!, so logged-in only.
maebeale
commented
Jun 15, 2026
| @@ -1,4 +1,7 @@ | |||
| Rails.application.routes.draw do | |||
| # Internal design reference for comparing page-header treatments | |||
| get "styleguide/headers", to: "styleguide#headers" | |||
Collaborator
Author
There was a problem hiding this comment.
🤖 From Claude: Reference/spec route only — meant to be removed (or moved behind an admin constraint) once the header system is chosen and the real shared partials land.
Introduce an internal /styleguide/headers page so we can compare every candidate page-header treatment side by side and converge on a consistent system before building shared partials and sweeping the site. The audit found ~5 drifting header variants across 500+ templates. This reference renders them as labelled families (list, charcoal+stripe topper, centered hero, public gradient) with index/show/new-edit states, plus the back-link, subnav, and action-placement conventions, so the look-and-feel decisions can be made visually rather than in the abstract. Logged-in only; no model data required, so authorization is skipped. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
976f075 to
5d837b3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is the goal of this PR and why is this important?
How did you approach the change?
GET /styleguide/headers(StyleguideController#headers) — logged-in only, no model data, authorization skipped.app/views/styleguide/headers.html.erbplus two demo partials (_demo,_utility_row) that render every candidate treatment with sample content using the realDomainThemecolor tokens.admin_pathon admin pages; subnav right), actions in the title row.Anything else to add?
shared/_page_header+shared/_subnavand converts pages in batches.DomainThemecolors when the real partials land.