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/core docs #66

Merged
merged 5 commits into from
Sep 13, 2016
Merged

Feature/core docs #66

merged 5 commits into from
Sep 13, 2016

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Sep 12, 2016

Description

Updated rest of core docs with ngModule.
#65

What's included?

  • Updated stepper docs with ngModule and removed directives from examples.
  • Updated expansion docs and removed directives from examples.
  • Updated media docs and removed directives/providers from examples.

Test Steps

  • ng serve
  • Go to docs.
  • 🎉
Screenshots or link to CodePen/Plunker/JSfiddle

image

<p>Import the [CovalentCoreModule] using the <code>forRoot()</code> method in your NgModule:</p>
<td-highlight lang="typescript">
<![CDATA[
import { CovalentCoreModule } from '@angular/core';
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be @covalent/core?

Copy link
Contributor Author

@emoralesb05 emoralesb05 Sep 12, 2016

Choose a reason for hiding this comment

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

damn it, copy paste is my worse enemy haha

@emoralesb05 emoralesb05 added this to the Alpha 0.7 milestone Sep 12, 2016
@kyleledbetter
Copy link
Contributor

is TdMediaToggleDirective still a directive that's usable? can you add docs to use it in here?

@emoralesb05
Copy link
Contributor Author

Its there in the docs

<md-card-title>TdMediaToggleDirective</md-card-title>

@kyleledbetter kyleledbetter merged commit 98f1ce8 into develop Sep 13, 2016
@emoralesb05 emoralesb05 deleted the feature/core-docs branch September 13, 2016 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants