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

solved issue #1534 and changed navbar UI #1550

Closed
wants to merge 1 commit into from
Closed

solved issue #1534 and changed navbar UI #1550

wants to merge 1 commit into from

Conversation

AlaynaShaheen
Copy link

Description

Contact Us in the navbar appeared 2 times and also there is no proper spacing between elements of Navbar. So I have added proper spacing in the navbar and enhanced its UI.

Related Issues

Type of PR

  • [*] Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)

[Attach any relevant screenshots or videos demonstrating the changes]

Screenshot 2024-08-02 185156

Checklist

  • [ X] I have gone through the contributing guide
  • [ X] I have updated my branch and synced it with project main branch before making this PR
  • [ X] I have performed a self-review of my code
  • [ X] I have tested the changes thoroughly before submitting this pull request.
  • [ X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • [ X] I have commented my code, particularly in hard-to-understand areas.

Additional context:

[Include any additional information or context that might be helpful for reviewers.]

Copy link

github-actions bot commented Aug 2, 2024

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for foodies-ordering-website ready!

Name Link
🔨 Latest commit 4593f98
🔍 Latest deploy log https://app.netlify.com/sites/foodies-ordering-website/deploys/66ace101bcb31a000807c17b
😎 Deploy Preview https://deploy-preview-1550--foodies-ordering-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@AlaynaShaheen
Copy link
Author

@sunny0625 please check the PR and let me know if there are any further changes that can be made.
Also kindly add the level of gssoc in the issue #1534.

@sunny0625
Copy link
Collaborator

solve conflicts.

@AlaynaShaheen AlaynaShaheen closed this by deleting the head repository Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc GSSOC'24 issues level1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contact Us in the navbar appeared 2 times and also there is no proper spacing between elements of Navbar
2 participants