Skip to content

Refactor navigation to use bootstrap navbar and jasny navmenu#41

Merged
robertsoniv merged 14 commits intoordercloud-api:developmentfrom
spencerwalker:development
Apr 18, 2017
Merged

Refactor navigation to use bootstrap navbar and jasny navmenu#41
robertsoniv merged 14 commits intoordercloud-api:developmentfrom
spencerwalker:development

Conversation

@spencerwalker
Copy link
Copy Markdown
Contributor

This is still a WIP but the basic foundation for a simplified nav UI is in place

  • Refactor and simplify custom navigation
  • Use navmenu on smaller viewports
  • Simplify typeahead UI
  • Clean out some unused LESS

OC-2054

* Replace custom nav elements with default bootstrap nav
* Use jasny-bootstrap navmenu on smaller viewports
* Add class for hiding certain nav elements on smaller viewports

OC-2054
…ar-buyer into development

# Conflicts:
#	src/app/base/templates/navigation.html
#	RESOLVED BY: added new ui-sref to "Orders" anchor tags

OC-2054
…ar-buyer into development

* Remove sticky behavior from category nav list

OC-2057
* Remove icons where not needed
* Add ordercloud search component to nav
* Add TODO for hiding/showing nav items

OC-2054
* Add template for typeahead popup
* Move templates for matches and popup to separate
file
* Use default bootstrap input-group for search
input

OC-2129
* Add mobile search functionality
* Delete commented out code
* Add class for hiding nav items on desktop
* Fix white-space bug on typeahead match
Copy link
Copy Markdown
Contributor

@robertsoniv robertsoniv left a comment

Choose a reason for hiding this comment

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

Nice!

@robertsoniv robertsoniv merged commit 37a82a4 into ordercloud-api:development Apr 18, 2017
crhistianramirez pushed a commit to culrich11/angular-buyer that referenced this pull request May 2, 2017
robertsoniv added a commit to robertsoniv/angular-buyer that referenced this pull request May 18, 2017
…sign_sw

CCN-171 modal, productsearch and typeahead less components
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.

2 participants