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

Update and Enhancement of UI #804

Merged

Conversation

KRISHkg31
Copy link
Contributor

Updation: Removal of the Dark Mode toggle button, as the website’s UI is already in black and white. Keeping the Dark Mode button would reduce the visibility of the website logo and complicate other design elements.

After the Changes:-
image

Enhancement :-

  1. There were a few issues with the UI. The first one was that the visitor counter was not properly positioned and was overlapping the LinkedIn and GitHub logos, so I fixed its placement.

Before the Changes:-
image

After the Changes:-
image

  1. The second issue was on the Contact Us page, where two 'scroll to top' buttons appeared when the user scrolled down the page. I have fixed this as well.

Before the Change:-
image

After the Change:-
image

Copy link

vercel bot commented Nov 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
trend-trove-ecommerce ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 10, 2024 9:30am

@KRISHkg31
Copy link
Contributor Author

@Tejashri-Taral I have enhance the Vsitor count button and also done some addition in the Dark Mode button and Scroll to top button please have a look and let me know if anything else needed. Request you to please merge the PR.
Thank You

@Tejashri-Taral
Copy link
Owner

Mention issue no.

@KRISHkg31
Copy link
Contributor Author

Issue No. - #807

@Tejashri-Taral
Copy link
Owner

Please resolve the conflicts

@KRISHkg31
Copy link
Contributor Author

@Tejashri-Taral Resolved all the conflicts

@Tejashri-Taral Tejashri-Taral merged commit 6b1cbf9 into Tejashri-Taral:main Nov 10, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants