Skip to content

docs: allow brand ring as location in choice→consequence rule - #49

Open
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/fix-docs/docs-allow-brand-ring-as-location-in-choice-conseq-b09865
Open

docs: allow brand ring as location in choice→consequence rule#49
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/fix-docs/docs-allow-brand-ring-as-location-in-choice-conseq-b09865

Conversation

@detail-app

@detail-app detail-app Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

§7.2 of design-console-ux.md ("Choice→consequence editors") still said
"a risky option must never wear the safe hue" after commit 807ae84 switched
the choice-card selection ring from ring-white/40 to ring-brand-500/50.
That rule now literally contradicts the implementation: the risky self-approval
option in policies_live.ex approval-requirements choice cards wears the
emerald brand ring when selected. Clarified that the brand ring marks the active
input as location (not a verdict); green fills and status treatments remain
forbidden — matching the model the same commit already applied to §2 of this
file and to design-complex-editors-follow-decision-order.md.

Type of Change

  • Documentation

Checklist

  • I have read LICENSE.md.
  • I have read CONTRIBUTING.md.
  • I have the right to submit this contribution.
  • This contribution does not include confidential information.
  • This contribution does not include secrets, credentials, private keys, tokens, or production data.
  • This contribution does not include third-party code without compatible licensing and attribution.
  • I have disclosed material AI assistance if applicable.
  • Tests and documentation are updated where appropriate.

AI Assistance Disclosure

Did you use AI assistance materially in this contribution?

  • Yes — details below
Detail: Doc Drift

Additional Notes

Introduced by commit 807ae84 (@AndrewDryga)


Doc Drift PRs can be configured here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant