Skip to content

Show complete proposal summaries in uniform cards - #74

Merged
Cyber-preacher merged 1 commit into
masterfrom
fix/proposal-card-full-summaries
Sep 4, 2026
Merged

Show complete proposal summaries in uniform cards#74
Cyber-preacher merged 1 commit into
masterfrom
fix/proposal-card-full-summaries

Conversation

@Cyber-preacher

Copy link
Copy Markdown
Contributor

Why

Proposal cards shortened summary strings in JavaScript and then clipped them to one CSS line. Complete descriptions such as the Ember proposal were therefore invisible, while allowing unrestricted wrapping would make card heights inconsistent.

What changed

  • preserve the complete normalized summary in the shared GlassyRecordCard
  • place summaries in a full-width responsive row with stable card geometry
  • reserve consistent title and summary regions across desktop, tablet, and mobile
  • retain contained scrolling only for exceptional legacy text beyond the reserved region
  • add regression coverage for complete summary rendering and the stable layout contract

Because this is implemented in the shared record card, proposal lists, chamber records, Feed records, and public drafts use the same behavior.

Validation

  • yarn test - 54 files, 240 tests passed
  • yarn typecheck
  • yarn build
  • Prettier and git diff --check
  • browser verification against all 13 public production proposals at 390, 768, 1024, and 1440 CSS pixels: equal card heights with no title or summary overflow

Data safety

Web-only presentation change. No API contract, migration, runtime mock wiring, production data, or preview artifact is included.

@Cyber-preacher
Cyber-preacher merged commit 69bd8d5 into master Sep 4, 2026
11 of 12 checks passed
@Cyber-preacher
Cyber-preacher deleted the fix/proposal-card-full-summaries branch September 4, 2026 12:47
Cyber-preacher added a commit that referenced this pull request Sep 4, 2026
Revert "Show complete proposal summaries in uniform cards (#74)"

This reverts commit 69bd8d5.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant