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 "activePage" as a active class in navigation menu #1335

Closed
wants to merge 4 commits into from

Conversation

psdwebdesigns
Copy link
Contributor

activePage is a active class which will be applied when any page is active. For example. If you visit any category page then this "activePage" class will be activated and that particular category name will be highlighted in different color.

@bigbot
Copy link

bigbot commented Aug 25, 2018

Autotagging @bigcommerce/storefront-team @davidchin

@bigmadness
Copy link

active for pages?

@psdwebdesigns
Copy link
Contributor Author

Hello @bigmadness

For web pages, i am working on it and soon i will submit that too to cornerstone team.

@psdwebdesigns
Copy link
Contributor Author

@bigmadness

I have added the activePage class too for web pages. Please check.

@psdwebdesigns
Copy link
Contributor Author

@karen-white @junedkazi

can anyone please let me know whether my changes are merged in cornerstone or not. Can i close this pull request if it is merged or not.

@karen-white
Copy link

@taruntds Thanks for submitting this! Our team will review the changes as soon as they have a moment. Once it's reviewed, we'll comment if there are any questions, or merge it, in which case the pull request would close automatically.

Copy link
Contributor

@junedkazi junedkazi left a comment

Choose a reason for hiding this comment

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

Overall it looks good. Can you please attach some screenshot of how it looks before & after would be nice. Also can you please add an entry in the changelog file.

@Ubersmake
Copy link
Contributor

Commits 31365c7 and 6432726 were squashed and pushed through with #1354

@Ubersmake Ubersmake closed this Sep 24, 2018
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.

6 participants