-
Notifications
You must be signed in to change notification settings - Fork 3
Added button and navbar components #18
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
base: main
Are you sure you want to change the base?
Conversation
Mobile menu may still need some work if we want it to look different to how it looks on desktop. Other than that, I think everything is there |
I've just pushed a commit with the missing dependencies and I've taken another look at how things are functioning. I'm finding that when I click "Projects" in the navbar, the menu opens fine. It's when I click the down arrow that it just turns around and does nothing. I'm going to hold off from accepting the PR until we can work out why this is happening. |
I've made a minor change to the selector for detecting areas to click-off. For some reason, it looks like the chevron navigation menu icon was being seen as a separate object and was causing the site to think I was clicking off the menu. Happy to merge this if you are happy with how the nav menu is looking. |
Happy with how it looks, merge it. |
Keeping the PR open to allow time for the mobile navbar to be improved. |
No description provided.