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

🐞[bug]: Changing the styling of navbar and order online and collect order cards #505

Closed
4 tasks done
Kritika745 opened this issue Oct 13, 2024 · 7 comments · Fixed by #511
Closed
4 tasks done
Assignees
Labels

Comments

@Kritika745
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

I think that the navbar may look much better if we work on its styling the logo is overflowing on hover and the navlinks have border very below them on hover which is not looking good.
we can decrease the length of the navbar making it sheek.
The two cards below for online order and takeaway are very larger in size which is not required as the content is very less.

Expected behavior

the look of the website will get enhanced and user experience will be good.

Add ScreenShots

Screenshot 2024-10-13 200727
Screenshot 2024-10-13 200740
Screenshot 2024-10-13 200817

What browsers are you seeing the problem on?

Chrome

Record

  • I have read the Contributing Guidelines
  • I'm a GSSOC'24EXTD contributor
  • I have starred the repository
Copy link

Hi there! Thanks for opening this issue @Kritika745!. We appreciate your contribution to this open-source project. We aim to respond or assign your issue as soon as possible.

@Anjaliavv51
Copy link
Owner

@Kritika745 navbar changes not required , change only for those order boxes.

@Kritika745
Copy link
Contributor Author

okay thank you. Can i make those boxes more attractive by adding some images or icons

@Kritika745
Copy link
Contributor Author

Also I have a question if you don't mind me asking,Is there a criterion according to which you merge pr because you have over 40 pr pending ?

@Anjaliavv51
Copy link
Owner

Also I have a question if you don't mind me asking,Is there a criterion according to which you merge pr because you have over 40 pr pending ?

No , had some work , today i had gone max to all , but still pending.

@Kritika745
Copy link
Contributor Author

Hey @Anjaliavv51 I have solved this issue under pr #511 please review it and merge it.

@Kritika745
Copy link
Contributor Author

hey @Anjaliavv51 please review it and merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants