Skip to content

feat(subscribe-cta): wire Tooltip to icon-only Share/Bookmark buttons - #826

Merged
greatest0fallt1me merged 3 commits into
CalloraOrg:mainfrom
Legit003:feat/subscribe-cta-icon-tooltips
Jul 29, 2026
Merged

feat(subscribe-cta): wire Tooltip to icon-only Share/Bookmark buttons#826
greatest0fallt1me merged 3 commits into
CalloraOrg:mainfrom
Legit003:feat/subscribe-cta-icon-tooltips

Conversation

@Legit003

Copy link
Copy Markdown
Contributor
  • Add Share and Bookmark icon-only action buttons to SubscribeCTA sticky bar
  • Wrap each button in the shared Tooltip primitive with hover-delay and long-press support (tooltipHoverDelayMs=300, tooltipLongPressMs=500)
  • New props: onShare, onBookmark, isBookmarked, tooltipHoverDelayMs, tooltipLongPressMs — all existing props are unchanged
  • Default share action copies current URL via navigator.clipboard
  • Bookmark toggles aria-pressed + active CSS class, fires onBookmark(bool)
  • WCAG 2.1 AA: aria-label on every button, aria-pressed on bookmark, role=group on actions container, aria-hidden SVGs, focus-visible ring
  • Add .subscribe-cta-bar__icon-btn styles with design tokens (light + dark)
  • 28 tests passing (25 new); no regressions in Tooltip or HelpPopover suites
  • Docs: docs/SubscribeCTA-icon-buttons-tooltip.md

GrantFox FWC26 campaign
closes #706

- Add Share and Bookmark icon-only action buttons to SubscribeCTA sticky bar
- Wrap each button in the shared Tooltip primitive with hover-delay and
  long-press support (tooltipHoverDelayMs=300, tooltipLongPressMs=500)
- New props: onShare, onBookmark, isBookmarked, tooltipHoverDelayMs,
  tooltipLongPressMs — all existing props are unchanged
- Default share action copies current URL via navigator.clipboard
- Bookmark toggles aria-pressed + active CSS class, fires onBookmark(bool)
- WCAG 2.1 AA: aria-label on every button, aria-pressed on bookmark,
  role=group on actions container, aria-hidden SVGs, focus-visible ring
- Add .subscribe-cta-bar__icon-btn styles with design tokens (light + dark)
- 28 tests passing (25 new); no regressions in Tooltip or HelpPopover suites
- Docs: docs/SubscribeCTA-icon-buttons-tooltip.md

GrantFox FWC26 campaign
@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@Legit003 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@greatest0fallt1me

Copy link
Copy Markdown
Contributor

Merged into main via admin resolver (-X theirs).

@greatest0fallt1me
greatest0fallt1me merged commit 311764b into CalloraOrg:main Jul 29, 2026
3 checks passed
@greatest0fallt1me

Copy link
Copy Markdown
Contributor

Green across the board — merging. Thanks!

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.

Add tooltip primitive usage on SubscribeCTA icon buttons [b#029]

2 participants