Param302 / SocialRepo

Social Repo - Your Social Handles Repository in a chrome extension.

Apache License 2.0

39 stars 35 forks source link

Added Confirmation Pop ups for copied links #72

Closed Max3796 closed 1 week ago

Max3796 commented 1 week ago

Popup for the confirmation if copied

When we click on the app we were not able to see if the link is copied or not so for that I put the confirmation popup coming

Issue Ticket Number

Fixes #56

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

Max3796 commented 1 week ago

@CodiumAI-Agent /review

CodiumAI-Agent commented 1 week ago

PR Reviewer Guide 🔍

⏱️ Estimated effort to review [1-5]	2
🧪 Relevant tests	No
🔒 Security concerns	No
⚡ Key issues to review	Possible Bug: The `showCopyMessage` function uses a hardcoded delay of 3000 milliseconds to hide the confirmation popup. Consider making this delay configurable or ensuring it aligns with user experience expectations.
⚡ Key issues to review	Code Quality: In the `createSocialLink` function, the logic for showing and hiding elements on mouse events could be refactored for better readability and maintainability. Consider separating concerns or simplifying the interaction logic.