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

initialise the navbar component and fix other styles #48

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Devesh21700Kumar
Copy link
Contributor

Pull request checklist

Please check if your PR 🚀 fulfills the following requirements:

  • Build (npm run start) was run locally and new changes were working correctly
  • Lint (npm run linter) has passed locally and any fixes were made for failures

Pull request type

Fixes styling and initialises Navbar

Please check the type of change your PR introduces:

  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)

Does this introduce a breaking change?

  • No

Checklist_User's_Behalf:

  • Ready to be merged
    Yes

Other information

Screenshot of the present initialized view

image

@Devesh21700Kumar Devesh21700Kumar changed the title initialise the navbar styles and fix other styles initialise the navbar component and fix other styles Apr 4, 2021
Copy link
Contributor Author

@Devesh21700Kumar Devesh21700Kumar left a comment

Choose a reason for hiding this comment

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

@ivanoats PR #47 might have some conflicts with this in App.tsx file

@Devesh21700Kumar
Copy link
Contributor Author

Pull request checklist

Please check if your PR 🚀 fulfills the following requirements:

  • Build (npm run start) was run locally and new changes were working correctly
  • Lint (npm run linter) has passed locally and any fixes were made for failures

Pull request type

Fixes styling and initialises Navbar

Please check the type of change your PR introduces:

  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)

Does this introduce a breaking change?

  • No

Checklist_User's_Behalf:

  • Ready to be merged
    Yes

Other information

Screenshot of the present initialized view

image

@ivanoats do check this one out. Thanks

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.

1 participant