-
Notifications
You must be signed in to change notification settings - Fork 285
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request]: Add Zoom animation on Hover for Footer icons #333
Comments
🙌 Thank you for bringing this issue to our attention! We appreciate your input and will investigate it as soon as possible ! |
@mansiruhil13 can I get to do that? Assign me! |
Thanks for assigning. Will start working on it. |
@mansiruhil13 Could you assign this to level2 since I will be working on the Home page as well as the Contact page. |
@mansiruhil13, It seems that you have updated the footer UI recently. It caused conflicts in my code. |
|
@mansiruhil13 You have not yet replied. So well I have done the updates on the Contact page. Here is a video: Contact.page-Icons.Hover.Animation.mp4The same will be done for the home page. Just read the previous comment and let me know which format you want the icons to be in. |
Also I noticed that the options are not responsive(the icons are not redirecting to the corresponding websites). Do you want me to work on that issue? If so, please let me know and assign the issue to level3 |
Please do let me know @mansiruhil13 so that I can proceed working on the home page icons. |
yes |
Ok, thanks for the update. Will be adding responsiveness, and changing the Home page icons to only logo and removing the texts. I had started working on another issue due to no response from your end. I will resume working on this after I am done working on the current issue. |
I will let you know about the updates of the work done. |
@mansiruhil13 you have merged some broken code. My code had a conflict and the site is broken |
@mansiruhil13 I am done with the code but cannot generate a PR due to the broken site from your end(merge conflict) I am attaching the updated website from my end. I went ahead and added the hover animation to the social icons on all the pages. Social.Icons.mp4 |
@mansiruhil13 I have noticed that you have fixed the site, but again changed the UI of the site(footer). Seems like I have to start working from scratch again. Well no issues! Will go for one more time. |
yes there are few changes, and some of them assigned recently. so do changes in your code according to that. |
Yup it will take some time because I have to again start by resolving all the new merge conflicts. Will update you when I am done |
I must say that I have faced the most number of merge conflicts while working on this issue. Everyone seems to be interested in the footer section right now 😂 . Well this is also helping me practice out with merge conflicts. |
✅ This issue has been closed. Thank you for your contribution! If you have any further questions or issues, feel free to raise them, and we can discuss more! |
Thanks for reviewing the PR! |
Is there an existing issue for this?
Feature Description
Hover Animations on Footer Icons on Home page
Hover Animations on Footer Icons on Contact Us page
Use Case
The users will find it attractive and the usability of the site will increase, since the icons will be visually appealing.
Benefits
This will make the website more interactive.
Priority
High
Record
The text was updated successfully, but these errors were encountered: