-
Notifications
You must be signed in to change notification settings - Fork 0
[$200] Navigation Bar Should Match Current #419
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
Comments
Challenge https://www.topcoder.com/challenges/3e9e44c2-739d-47ce-a5fa-9df3dbd051e6 has been created for this ticket. |
Challenge https://www.topcoder.com/challenges/3e9e44c2-739d-47ce-a5fa-9df3dbd051e6 has been assigned to obog. |
@jmgasper a few questions about navigation. All links now navigate to pages in Vanilla. Should these links navigate to Topcoder pages or not?
|
|
Eventually, we're going to move the settings to Topcoder and have Topcoder make API calls back to Vanilla, but that's not on the current roadmap. |
@jmgasper What about nav bar on mobile? Is in the scope of the task? Topcoder sites: The menu opens in full screen, |
@atelomycterus - We can leave that for a future task for now |
@jmgasper Keep you updated: |
Great, thanks! |
@jmgasper Please apply PRs Thanks! TestingPlease clear browser cache before testing: If count of New Notifications > 0 then a dot should be displayed: |
Payment task has been updated: https://www.topcoder.com/challenges/3e9e44c2-739d-47ce-a5fa-9df3dbd051e6 |
The Navigation Bar should be the same as what is currently used on the Community Site. This is an issue we had before where if we made any change to the navigation on the Community site, we would have to make the same updates in Jive. I hope that will not be the case again.
Here is how it shows in dev currently:

This is how it should operate:

The text was updated successfully, but these errors were encountered: