-
Notifications
You must be signed in to change notification settings - Fork 215
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
Hamurger and progress bar implementation #459
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better
✅ Deploy Preview for arcticdelights ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Please merge my pull request. I have resolved the conflicts. |
Kindly resolve conflicts. |
I have successfully resolved all the conflicts |
I think there are still some conflicts in index.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kindly fix the following issues:
- In light mode, the text color is almost unreadable.
- We're unable to see the links after blogs. The scroll bar scrolls down the website below the hamburger menu as indicated by the progress bar above. But I can't scroll down the menu to see my orders and contact us page.
- If possible, showcase the cart, wishlist and signup options as we're currently unable to see them as there is no scroll for the menu.
Kindly resolve conflicts. |
ArcticDelights.-.Google.Chrome.2024-10-17.21-55-10.mp4i have done the changes that mentioned above also solved all the conflicts. |
Please merge |
This is the PR i have raised and it's not been merged and it's level 3 PR. could you review and merge it! dohinaf/basic-icecream-website#459
This PR has been merged by the GSSoC team Don't add labels to it now. Close this PR |
#385
Dear @dohinafs ,
I hope this message finds you well. I wanted to share that I have successfully implemented a hamburger menu for small screen sizes and a progress bar the project.
If you have any suggestions for further improvements or areas I should focus on next, please let me know.
Thank you!
Before:
NO HAMBURGER AND PROGRESS BAR
After:
HAMBURGER AND PROGRESS BAR