Skip to content

feat(animate): cross-browser compatibility#4243

Closed
marclaval wants to merge 2 commits into
angular:masterfrom
marclaval:fixAnimation
Closed

feat(animate): cross-browser compatibility#4243
marclaval wants to merge 2 commits into
angular:masterfrom
marclaval:fixAnimation

Conversation

@marclaval
Copy link
Copy Markdown
Contributor

Follow up of #4219, only the last 2 commits have to be reviewed.

@rkirov
Copy link
Copy Markdown
Contributor

rkirov commented Sep 17, 2015

let me know when it is ready for review

@marclaval marclaval force-pushed the fixAnimation branch 6 times, most recently from 51e5cd2 to ab32c35 Compare September 18, 2015 11:57
@marclaval marclaval added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: WIP labels Sep 18, 2015
@marclaval
Copy link
Copy Markdown
Contributor Author

Ready for review

/cc @robertmesserle

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It would be nice if we can use a node module or a github repo, in case someone fixes some bugs on it, etc. No worries if you can't find any.

@rkirov rkirov assigned robertmesserle and unassigned rkirov Sep 18, 2015
@rkirov
Copy link
Copy Markdown
Contributor

rkirov commented Sep 18, 2015

I thought this also broke during the traceur removal. Now I see that it is independent issue with animation, so I will let @robertmesserle do the final review. LGTM from my standpoint.

@robertmesserle
Copy link
Copy Markdown

LGTM

@marclaval marclaval closed this in bffa2cb Sep 24, 2015
robwormald pushed a commit to robwormald/angular that referenced this pull request Sep 25, 2015
@marclaval marclaval deleted the fixAnimation branch December 11, 2017 09:57
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants