Added the Navbar Routes Functionlaity #22
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the assignment assigned to me, I done all the task associated with Navbar routes functionality.
when user clicks on list items at the navbar like Logo, Home , Signup, Login etc , It renders to that particulars component on clicking without re-loading the page.
Also added the functionality to the Already have an acount **Sign In** in sign up form , when user clicks on the signIn button , it renders the Login page.
Same to the Did'nt have an acount **Sign Up** in login page, when user clicks on the Sign Up button , it renders the Signup page.
Note:
<div style={{ height: '100vh', overflowY: 'scroll', scrollbarWidth: 'thin', scrollbarColor: '#d3d3d3 #f1f1f1', }}>
I have to remove this Inline css in App.jsx as it is browser specific and it works properly only in Firefox, not in any other browser, also it affects the scroll Top feature when user clicks on Already have an acount **Sign In** .
Please Merge this Pull Request under Hacktoberfest and hacktoberfest-accepted label by reviewing the code functionality and features. @abhishekraoas
Thanks...