Skip to content

[WIP] Fix share buttons to open in new windows and update LinkedIn link#5

Merged
liampmccabe merged 2 commits intomainfrom
copilot/fix-share-buttons-functionality
Jan 15, 2026
Merged

[WIP] Fix share buttons to open in new windows and update LinkedIn link#5
liampmccabe merged 2 commits intomainfrom
copilot/fix-share-buttons-functionality

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 15, 2026

Fix Share Buttons Functionality

  • Research correct LinkedIn sharing URL and parameters
  • Update LinkedIn share URL to include title and fix preview issue
  • Add click event handlers to open share links in popup windows instead of new tabs
  • Ensure Twitter/X shares include the title properly in the tweet text
  • Update Facebook share URL if needed to include title
  • Test all share buttons to verify popup behavior and content sharing
  • Fix window name to be unique per platform for simultaneous sharing
  • Run code review and address feedback
  • Run security check
Original prompt

On index.js, Can you make the share buttons open in a little windows rather than changing the current page. Also LinkedIn doesn't seem to work: "Cannot display preview. You can post as is, or try another link."

Also for X (and presumably other sites), I just see the url, can we add the title too when sharing.


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

Co-authored-by: liampmccabe <3018957+liampmccabe@users.noreply.github.com>
@liampmccabe liampmccabe marked this pull request as ready for review January 15, 2026 09:29
@liampmccabe liampmccabe merged commit 5463c09 into main Jan 15, 2026
Copilot stopped work on behalf of liampmccabe due to an error January 15, 2026 09:30
Copilot AI requested a review from liampmccabe January 15, 2026 09:30
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