Skip to content

Navbar changed#126

Merged
Chris5613 merged 4 commits intoChris5613:mainfrom
ynzulak:main
Jun 11, 2023
Merged

Navbar changed#126
Chris5613 merged 4 commits intoChris5613:mainfrom
ynzulak:main

Conversation

@ynzulak
Copy link
Contributor

@ynzulak ynzulak commented Jun 7, 2023

I improved a Navbar with media queries and an animation. I only worked for 768px media queries here. I'm pulling request with a question. what do you think about making media queries by me? I can add some burger menu to it. Or even i can work for whole site. What do you think about that? Have a nice evening 😁.

Copy link
Owner

@Chris5613 Chris5613 left a comment

Choose a reason for hiding this comment

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

If you can add a hamburger menu that would be great, I will test your code when I'm free

@Chris5613
Copy link
Owner

@ynzulak you're going to need to make it so it fits on a moblie screen other than that it looks pretty good

Copy link
Owner

@Chris5613 Chris5613 left a comment

Choose a reason for hiding this comment

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

Are you testing the code before you submit? the hamburger menu does not show.

@Chris5613 Chris5613 linked an issue Jun 10, 2023 that may be closed by this pull request
@ynzulak
Copy link
Contributor Author

ynzulak commented Jun 10, 2023

I added a working burgermenu but without animations because i dont have enough time
Is it good?

Copy link
Owner

@Chris5613 Chris5613 left a comment

Choose a reason for hiding this comment

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

Looks good enough

@Chris5613 Chris5613 merged commit 659d593 into Chris5613:main Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New Navbar Needed

2 participants