Param302 / SocialRepo

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

Apache License 2.0

39 stars 35 forks source link

Updated styles of Hover effect #73

Closed Max3796 closed 1 week ago

Max3796 commented 1 week ago

Hover over the text to get app name

As only the app images were visible but to solve this issue I have used the hover effect such that when you hover over that it will show name of the app

Issue Ticket Number

Fixes #65

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] 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]	3
🧪 Relevant tests	No
🔒 Security concerns	No
⚡ Key issues to review	Possible Bug: The hover effect logic might not be responsive or smooth on all devices, as it relies heavily on JavaScript to manipulate CSS properties directly.
⚡ Key issues to review	Performance Concern: The frequent DOM manipulation on hover events (showing and hiding elements) might cause performance issues on devices with lower specs or with a high number of social links.