Fix case-study share chrome, hero lockup width, and copy confirmation in the share dialog. #75

Merged
an.di merged 1 commits from adilallo/fix/CR-212-case-studies-share into main 2026-09-10 14:48:49 +00:00
Owner

Overview

Case-study completed-rule pages showed a translation key instead of Share, squeezed the above-the-fold title into the thumbnail column, and the share dialog never confirmed a copied link. This restores the Figma lockup and share states, including on published rules that reuse the same dialog.

Changes

  • Marketing message bundle now includes create-flow top-nav copy so the case-study Share button reads Share
  • Case-study banner uses a full-width 365px hero lockup (44px type) instead of the 200px thumbnail width
  • Share dialog: hover and press on channel tiles; Copy link becomes a green check and Copied!; screen-reader announcement for the clipboard copy
  • Tests for the Share label, use-case container size, and in-dialog copy confirmation

Screenshots

How to Test

  1. npm run dev and open /use-cases/mutual-aid-colorado
  2. Confirm the hero title wraps on two lines in a ~365px column, and the author/date stay on one line
  3. Open /use-cases/mutual-aid-colorado/rule and confirm the top-nav button is Share, not create.topNav.share
  4. Open the share dialog: hover a tile (border/brightness), press it (slight shrink)
  5. Click Copy link — the tile should switch to a green check and Copied!
  6. Repeat Copy link from a published-rule share dialog if you have one — same confirmation
  7. Optional: npx tsc --noEmit and npx vitest run tests/components/modals/Share.test.tsx tests/components/ContentBanner.test.tsx tests/unit/ContentContainer.test.jsx tests/pages/use-cases-completed-rule.test.jsx

Notes

  • Slack, Discord, and Signal on case-study pages still only copy the URL and show the page banner behind the dialog. Opening those apps is wired on published rules, not on this template.
  • Copy link still also raises the existing page banner; in-dialog Copied! is the confirmation this change adds.
## Overview Case-study completed-rule pages showed a translation key instead of **Share**, squeezed the above-the-fold title into the thumbnail column, and the share dialog never confirmed a copied link. This restores the Figma lockup and share states, including on published rules that reuse the same dialog. ## Changes - Marketing message bundle now includes create-flow top-nav copy so the case-study Share button reads **Share** - Case-study banner uses a full-width 365px hero lockup (44px type) instead of the 200px thumbnail width - Share dialog: hover and press on channel tiles; Copy link becomes a green check and **Copied!**; screen-reader announcement for the clipboard copy - Tests for the Share label, use-case container size, and in-dialog copy confirmation ## Screenshots ## How to Test 1. `npm run dev` and open `/use-cases/mutual-aid-colorado` 2. Confirm the hero title wraps on two lines in a ~365px column, and the author/date stay on one line 3. Open `/use-cases/mutual-aid-colorado/rule` and confirm the top-nav button is **Share**, not `create.topNav.share` 4. Open the share dialog: hover a tile (border/brightness), press it (slight shrink) 5. Click **Copy link** — the tile should switch to a green check and **Copied!** 6. Repeat Copy link from a published-rule share dialog if you have one — same confirmation 7. Optional: `npx tsc --noEmit` and `npx vitest run tests/components/modals/Share.test.tsx tests/components/ContentBanner.test.tsx tests/unit/ContentContainer.test.jsx tests/pages/use-cases-completed-rule.test.jsx` ## Notes - Slack, Discord, and Signal on case-study pages still only copy the URL and show the page banner behind the dialog. Opening those apps is wired on published rules, not on this template. - Copy link still also raises the existing page banner; in-dialog **Copied!** is the confirmation this change adds.
an.di added 1 commit 2026-09-10 14:48:29 +00:00
Marketing pages omitted create-flow nav copy, so Share rendered as a translation key; the case-study banner squeezed hero text to the thumbnail width; Copy link had no hover or copied state.

Co-authored-by: Cursor <cursoragent@cursor.com>
an.di self-assigned this 2026-09-10 14:48:35 +00:00
an.di merged commit 95e1937419 into main 2026-09-10 14:48:49 +00:00
an.di deleted branch adilallo/fix/CR-212-case-studies-share 2026-09-10 14:48:50 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CommunityRule/community-rule#75