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.

Sign up for GitHub

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

Jump to bottom

Added Responsive Navbar #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Added Responsive Navbar #42

wants to merge 1 commit into from

Conversation

sahilg861907
Copy link

image

Abirpal202049
Abirpal202049 requested changes Mar 29, 2022
View reviewed changes
Copy link
Member

@Abirpal202049 Abirpal202049 left a comment

Choose a reason for hiding this comment

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

Sorry to say, but we can't approve it. You have imported a library and used its components which is quite easy. Our main purpose of this project is to make some good designer components, But you have just copied it, and it's similar to the bootstrap design. So we can't approve it until your further changes

@sahilg861907
Copy link
Author

I'll surely work on it. thank you

@Jaideep25-tech Jaideep25-tech added GSSoC22 Issues with this label is specific for GSSOC 2022 participants. Level2 GSSoC 22 Level 2 - 25 Points labels Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Abirpal202049 Abirpal202049 Abirpal202049 requested changes

Assignees
No one assigned
Labels
GSSoC22 Issues with this label is specific for GSSOC 2022 participants. Level2 GSSoC 22 Level 2 - 25 Points
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sahilg861907 @Abirpal202049 @Jaideep25-tech @sahilg125