Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

refactor(api): use Renderer2 instead of Renderer #372

Merged
merged 2 commits into from
Aug 10, 2017
Merged

Conversation

ThomasBurleson
Copy link
Contributor

  • Use Renderer2 instead of Renderer
  • Do not extend the Angular base directives, but instead inject them into the constructor.
  • improve support for future releases of Angular 5

Fixes #326.

@ThomasBurleson ThomasBurleson changed the title fix(api): use Renderer2 instead of Renderer refactor(api): use Renderer2 instead of Renderer Aug 9, 2017
@ThomasBurleson ThomasBurleson requested a review from crisbeto August 9, 2017 14:13
@ThomasBurleson
Copy link
Contributor Author

@crisbeto - Would you review plz?

* Use Renderer2 instead of Renderer
* Do not extend the Angular base directives, but instead inject them into the constructor.
* improve support for future releases of Angular 5

Fixes #326.
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@crisbeto crisbeto added pr: lgtm This PR has been approved by the reviewer and removed pr: needs review labels Aug 9, 2017
@mmalerba mmalerba merged commit 695bf37 into master Aug 10, 2017
@ThomasBurleson ThomasBurleson deleted the thomas/issue-326 branch September 13, 2017 22:13
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes pr: lgtm This PR has been approved by the reviewer pr: needs presubmit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flex-layout needs to upgrade to Renderer2
4 participants