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

Improved responsiveness of navbar #315

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pooja2299
Copy link

Fixes #288

Description

  1. navHidden variable was true initially. Made it false.- in Header.jsx file(line30).
  2. Added : display:none; - in media->nav class in header.module.scss file(line 74).
  3. Changed the condition at line 51 in Header.jsx

@pooja2299
Copy link
Author

Hi @gauravdas014 . There are some conflicts in yarn.lock file. Can you help in resolving them since i have no idea what dependencies(version of them) to be deleted.

@gauravdas014
Copy link
Collaborator

@pooja2299 @shashankmotruri will help you out with this.

Copy link
Collaborator

@shashankmotruri shashankmotruri left a comment

Choose a reason for hiding this comment

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

@pooja2299 delete yarn.lock and just run yarn again. And do follow contributing guidelines for commit message.

Copy link
Collaborator

@shashankmotruri shashankmotruri left a comment

Choose a reason for hiding this comment

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

@pooja2299 did u tried yarn ? .I think there are still conflicts with yarn.lock. Try yarn and then yarn install.And if conflits are cleared , do change commit message as mentioned in contributing.md.

@pooja2299
Copy link
Author

Yes , i tried yarn . But will try again and resolve conflict.

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

Successfully merging this pull request may close these issues.

Issue in responsiveness of navbar
3 participants