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

feature(deps): upgrade to @angular@2.4.1 and @material@beta.1 + initial AoT support. closes(#149) #209

Merged
merged 53 commits into from
Dec 28, 2016

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Dec 24, 2016

Description

Upgrade to @angular@2.4.1 and @Material@beta.1

Aot support #149

What's included?

Test Steps

  • rm -rf node_modules dist deploy tmp
  • npm i
  • ng serve
  • Play around with the docs to see everything working
  • npm run test and check /coverage/index.html

Additional Test Step

After http interceptor PR is merged.

  • ng serve --aot
  • See docs work in AoT.
  • npm run build for added AoT process

Screenshots

image

@emoralesb05 emoralesb05 added this to the Alpha 0.10 milestone Dec 24, 2016
@emoralesb05 emoralesb05 changed the title feature(deps): upgrade to @angular@2.4.1 and @material@beta.1 feature(deps): upgrade to @angular@2.4.1 and @material@beta.1 + AoT Docs Dec 25, 2016
@kyleledbetter
Copy link
Contributor

kyleledbetter commented Dec 27, 2016

I noticed an issue w/ nav-list and manage-list leftnav: If you click on the left nav items in desktop width first, then scale to mobile width, the toggle won't work. If you don't click on them in desktop width, they work as expected on mobile:

nav-list-leftnav

Fixes

  • nav-list toggle when clicked on desktop width
  • manage-list toggle when clicked on desktop width

@emoralesb05
Copy link
Contributor Author

  • Removed obsolete validation that clashed with style overrides.

@kyleledbetter kyleledbetter merged commit 6cd31f0 into develop Dec 28, 2016
@emoralesb05 emoralesb05 deleted the feature/latest-deps branch December 28, 2016 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants