Skip to content
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

feat: UI Improvements and Updates #78

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Conversation

itsharshitrwt
Copy link
Contributor

@itsharshitrwt itsharshitrwt commented Oct 21, 2024

Pull Request Update: Fixes #52

Hi @lcrojano ,
I am excited to share that I have made new UI changes to the Giphy website.

Previous:

Screenshot 2024-10-22 000720

Screenshot 2024-10-22 000758

New:

Screenshot 2024-10-22 000740

Screenshot 2024-10-22 000805

Additionally, I have improved the login and signup pages. Please note that I did not work on the "Add to Favorite" button, as that issue was assigned to someone else.

Suggestion: I recommend removing the website links from the GIF cards for a cleaner look.

If everything looks good, I would greatly appreciate it if you could take a moment to review and merge the PR.

Thank you!

Copy link
Owner

@lcrojano lcrojano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@itsharshitrwt Great contribution! 🎉

Before we proceed with accepting the PR, could you please pull the latest changes from the repository to ensure everything is up to date, especially in the ./husky folder? It looks like your current commit doesn’t include the latest updates from the main branch.

Thanks again for your hard work! 🚀

@itsharshitrwt
Copy link
Contributor Author

itsharshitrwt commented Oct 22, 2024

Hi @lcrojano, Thanks for reviewing , I have pulled out latest changes. I hope this solves the issue if there is still some problem plz let me know :)

If everything looks good, I would greatly appreciate it if you could take a moment to review and merge the PR 😄

Thank you!

@lcrojano lcrojano merged commit 1525028 into lcrojano:main Oct 22, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest hacktoberfest-accepted hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: Improve UI with [Modern UI Style] Design Guidelines
2 participants