Skip to content

Fix shareTitle prop typo in ShareButton component#27

Merged
AccessiT3ch merged 2 commits intotest/phase1.5-test-coveragefrom
copilot/sub-pr-26
Feb 1, 2026
Merged

Fix shareTitle prop typo in ShareButton component#27
AccessiT3ch merged 2 commits intotest/phase1.5-test-coveragefrom
copilot/sub-pr-26

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 1, 2026

The ShareButton component used shareTite (typo) instead of shareTitle, causing the title to be dropped when invoking mobile native share.

Changes:

  • Renamed prop from shareTite to shareTitle in ShareButton component and PropTypes
  • Updated all test expectations to use correct parameter name
  • Fixed ResourceModal caller to pass shareTitle

Before:

// ShareButton.jsx
function ShareButton({ shareUrl, shareTite, shareText }) {
  shareHandler({ shareUrl, shareTite, shareText });
  //                       ^^^^^^^^^ typo - utils.js expects shareTitle
}

After:

// ShareButton.jsx
function ShareButton({ shareUrl, shareTitle, shareText }) {
  shareHandler({ shareUrl, shareTitle, shareText });
  //                       ^^^^^^^^^^ now matches utils.js interface
}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: AccessiT3ch <95449364+AccessiT3ch@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback on establishing test coverage baseline Fix shareTitle prop typo in ShareButton component Feb 1, 2026
Copilot AI requested a review from AccessiT3ch February 1, 2026 20:13
@AccessiT3ch AccessiT3ch marked this pull request as ready for review February 1, 2026 20:16
@AccessiT3ch AccessiT3ch merged commit 74bbbc7 into test/phase1.5-test-coverage Feb 1, 2026
1 check passed
@AccessiT3ch AccessiT3ch deleted the copilot/sub-pr-26 branch February 1, 2026 20:17
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.

2 participants