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

feat(docs): Covalent Text/Markdown Editor Demo #804

Merged
merged 5 commits into from
Aug 9, 2017

Conversation

jeremysmartt
Copy link
Collaborator

Description

Added demo for Covalent Text/Markdown Editor into Covalent Docs

What's included?

  • new file: src/app/components/components/text-editor/text-editor.component.html
  • new file: src/app/components/components/text-editor/text-editor.component.scss
  • new file: src/app/components/components/text-editor/text-editor.component.ts
  • modified: package.json
  • modified: src/app/components/components/components.component.ts
  • modified: src/app/components/components/components.module.ts
  • modified: src/app/components/components/components.routes.ts
  • modified: src/app/components/components/overview/overview.component.ts
  • modified: src/app/components/toolbar/toolbar.component.ts

Test Steps

  • checkout branch
  • rm -rf node_modules
  • npm i
  • ng serve
  • go to http://localhost:4200/#/components
  • Click on Text/Markdown Editor under optional components
  • Try out editor and see output

General Tests for Every PR

  • ng serve --aot still works.
  • npm run lint passes.
  • npm test passes and code coverage is not lower.
  • npm run build still works.
Screenshots or link to CodePen/Plunker/JSfiddle

screen shot 2017-08-08 at 3 44 03 pm

jeremy.smartt added 3 commits August 8, 2017 15:39
@kyleledbetter
Copy link
Contributor

Pushed some updates to the demo in the docs:

image

@emoralesb05 emoralesb05 added this to the Beta.7 milestone Aug 9, 2017
@emoralesb05 emoralesb05 merged commit 3127d76 into develop Aug 9, 2017
@emoralesb05 emoralesb05 deleted the feature/text-editor-docs branch August 9, 2017 22:09
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.

3 participants