One button system: adopt the four roles and gate them in CI - #1377
Merged
Conversation
Converts 12 dialog buttons to the four roles the frontend defines — .btn-primary, .btn-cancel, .btn-delete, .btn-quiet. Every id, binding, label and button order preserved; only the Material directive and its inert `color` input are removed. Role follows the label and intent, not the old colour. Also adds the gate to both workflows, which is the part that keeps this fixed. The checker lives in the frontend repo, but a frontend CI run cannot see these templates — src/app/plugins/ is gitignored there — so without this step nothing checks outer-inner-resource-pn's buttons and they drift straight back. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015sXLtgzZU8QL9m84GqMkoJ
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
1 participant
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.
Plugin half of the button consolidation. Depends on microting/eform-angular-frontend#8016, already merged to
stable, which defines the roles and ships the checker.Audit: Button Drift
What changes
12 dialog buttons move to the four roles:
.btn-primary(filled),.btn-cancel(outlined),.btn-delete(destructive),.btn-quiet(text). Everyid, binding, tooltip, label and button order is preserved — only the Material directive and itscolorinput go, andcolorwas inert on a plain button anyway.Role follows the label and intent, not the old colour. Several confirms carried
color="warn"without being destructive and are.btn-primary.Why the CI step matters more than the conversion
The checker lives in the frontend repo, but a frontend CI run cannot see these templates —
src/app/plugins/is gitignored there, so its checkout contains only the core app. The frontend PR fixed 55 violations in core and was structurally incapable of seeing the 91 in the plugins.So each plugin repo has to run the checker over its own module, or nothing checks its buttons and they drift straight back. That step is added to both workflows here.
What it enforces
Inside a
mat-dialog-actionsrow, every button must carry one of the four roles; no template may reference a class no stylesheet defines. Material's button directives are rejected in both spellings — the theme override targeted.mat-mdc-text-button, which Angular Material 20 never emits, somat-buttonrendered as a pill in the wrong colour and always had.🤖 Generated with Claude Code
https://claude.ai/code/session_015sXLtgzZU8QL9m84GqMkoJ