Improve video sharing functionality and clean up page structure
Update share buttons to use divs instead of buttons for better accessibility, and adjust the header element in the home page component. Replit-Commit-Author: Agent Replit-Commit-Session-Id: 917af350-dda6-4e65-b67d-4f315b389291 Replit-Commit-Checkpoint-Type: full_checkpoint
This commit is contained in:
parent
84c43e3b9b
commit
5d18414f25
@ -387,22 +387,22 @@ export default function VideoPage() {
|
||||
{showShareMenu && (
|
||||
<div className="absolute right-0 top-full mt-2 bg-gray-800 rounded-lg shadow-lg py-2 z-50 min-w-[200px]">
|
||||
<FacebookShareButton url={getShareUrl()}>
|
||||
<button className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2">
|
||||
<div className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2 cursor-pointer">
|
||||
<FacebookIcon size={16} round />
|
||||
Facebook
|
||||
</button>
|
||||
</div>
|
||||
</FacebookShareButton>
|
||||
<TwitterShareButton url={getShareUrl()} title={`Watch "${currentVideo.title}" on go4.video`}>
|
||||
<button className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2">
|
||||
<div className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2 cursor-pointer">
|
||||
<TwitterIcon size={16} round />
|
||||
Twitter
|
||||
</button>
|
||||
</div>
|
||||
</TwitterShareButton>
|
||||
<WhatsappShareButton url={getShareUrl()} title={`Watch "${currentVideo.title}" on go4.video`}>
|
||||
<button className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2">
|
||||
<div className="w-full px-4 py-2 text-left text-white hover:bg-gray-700 flex items-center gap-2 cursor-pointer">
|
||||
<WhatsappIcon size={16} round />
|
||||
WhatsApp
|
||||
</button>
|
||||
</div>
|
||||
</WhatsappShareButton>
|
||||
<button
|
||||
onClick={copyToClipboard}
|
||||
|
||||
@ -98,7 +98,7 @@ export default function Home() {
|
||||
/>
|
||||
</div>
|
||||
</div>
|
||||
</header>
|
||||
</div>
|
||||
|
||||
<main className="w-full pt-0 pb-8 relative">
|
||||
|
||||
|
||||
Loading…
Reference in New Issue
Block a user