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

chore(): update documentation with the way to use media.broadcast (closes #684) #685

Merged
merged 1 commit into from
Jun 12, 2017

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Jun 10, 2017

angular is more restrictive now and complains if we dont explicitly call a change detection cycle when changing any input via a life hook cycle such as ngAfterViewInit and ngAfterContentInit..

this happens because the change detections have finished by then, so if you must change anything at that point, you need to call explicitly for a new change detection cycle via detectChanges

#684

Test Steps

  • Go and see layout docs like nav-list and manage-list referencing how to use media in the html example comments.

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.

…h layouts in ngAfterViewInit

angular is more restrictive now and complains if we dont excplicitly call a change detection cycle when changing any input via a life hook cycle such as ngAfterViewInit and ngAfterContentInit..

this happens because the change detections have finished by then, so if you must change anything at that point, you need to call explicitly for a new change detection cycle via `detectChanges`
emoralesb05 added a commit to Teradata/vantage-ui-template that referenced this pull request Jun 10, 2017
emoralesb05 added a commit to Teradata/vantage-ui-template that referenced this pull request Jun 10, 2017
* chore(): update quickstar with proper media.broadcast usage

reference Teradata/covalent#685

* fix(): ninja remove version from package.json
@emoralesb05 emoralesb05 changed the title chore(): update documentation with the way to use media.broadcast chore(): update documentation with the way to use media.broadcast (closes #684) Jun 10, 2017
@emoralesb05 emoralesb05 merged commit 2606b75 into develop Jun 12, 2017
@emoralesb05 emoralesb05 deleted the feature/media-layout-docs branch June 12, 2017 05:34
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.

1 participant