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

added active class #139

Merged
merged 2 commits into from
Oct 14, 2022
Merged

added active class #139

merged 2 commits into from
Oct 14, 2022

Conversation

biswajit035
Copy link
Contributor

@biswajit035 biswajit035 commented Oct 13, 2022

Describe your changes

I have added a ACTIVE class so that user can get idea about his active tab in NAVBAR

Screenshots - If Any (Optional)

AFTER

Issue ticket number and link -

#138

Checklist before requesting a review

  • I have performed a self-review of my code.

  • Followed the repository's Contributing Guidelines.

  • I ran the app and tested it locally to verify that it works as expected.

  • I have starred the repository.

Additional Information (Optional)

@devarshishimpi
Copy link
Member

@biswajit035 Kindly complete all steps as requested before review

Copy link
Contributor Author

@biswajit035 biswajit035 left a comment

Choose a reason for hiding this comment

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

I have reviewed the changes

Comment on lines +171 to +174
.active{
background-color: #e4e9f7;
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

for active class background color

@@ -89,7 +90,7 @@ const Sidenav = () => {
</a>
</li>

<li className="nav-link">
<li className={`nav-link ${location === '/bin' ? "active" : ""}`}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For automatically adding the active class according to the url

<Link to={'/notes'}>
<i className="fa-solid fa-house icon"></i>
<span className="text nav-text">Dashboard</span>
</Link>
</li>

<li className="nav-link">
<li className={`nav-link ${location === '/myaccount' ? "active" : ""}`}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For automatically adding the active class according to the url

@@ -68,14 +69,14 @@ const Sidenav = () => {
</li>

<ul className="menu-links">
<li className="nav-link">
<li className={`nav-link ${location === '/notes'?"active":""}`}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For automatically adding the active class according to the url

@biswajit035
Copy link
Contributor Author

This is my 2nd PR of my first OPENSOURCE, So I have no idea what to do now.
Can you please guide ?

@devarshishimpi
Copy link
Member

This is my 2nd PR of my first OPENSOURCE, So I have no idea what to do now. Can you please guide ?

That's it, it will be merged

Copy link
Member

@devarshishimpi devarshishimpi left a comment

Choose a reason for hiding this comment

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

LGTM!! Thanks For Contributing!!!

@devarshishimpi devarshishimpi merged commit e86a279 into bytemakers:main Oct 14, 2022
@devarshishimpi devarshishimpi added enhancement New feature or request v2.0-roadmap Version 2.0 Roadmap hacktoberfest Hacktoberfest hacktoberfest-accepted Hacktoberfest Accepted hacktoberfest2022 Hacktoberfest 2022 labels Oct 14, 2022
@biswajit035
Copy link
Contributor Author

That's it, it will be merged

It is merged but when I am going to https://noteslify.stonecss.com/notes this features is not working.
but I pulled the code and run localy it is working fine.
Can you please help to make this features live?

@devarshishimpi
Copy link
Member

Yes, you are right. The feature will be brought into production on the next release ( Maybe today or tomorrow )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request hacktoberfest Hacktoberfest hacktoberfest2022 Hacktoberfest 2022 hacktoberfest-accepted Hacktoberfest Accepted v2.0-roadmap Version 2.0 Roadmap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants