Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

fix(dialog): changed translate3d to translate #6729

Closed
wants to merge 1 commit into from
Closed

fix(dialog): changed translate3d to translate #6729

wants to merge 1 commit into from

Conversation

EladBezalel
Copy link
Member

fixes #4544

@EladBezalel EladBezalel added the needs: review This PR is waiting on review from the team label Jan 17, 2016
@ThomasBurleson
Copy link
Contributor

@EladBezalel - See http://stackoverflow.com/questions/22111256/translate3d-vs-translate-performance.
So I think for only IE should we use translate (instead of translate3d). Have we analyzed the perf impact on Chrome when switch to translate(...)?

@EladBezalel
Copy link
Member Author

Ran the jsprdf both on my pc and phone with chrome and both showed better results with translate..

@ThomasBurleson
Copy link
Contributor

@EladBezalel - sgtm. thx.

@ThomasBurleson ThomasBurleson deleted the fix/dialog-blurry-text branch January 19, 2016 04:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
needs: review This PR is waiting on review from the team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

md-dialog: IE11 / Edge blurry text and buttons after animating onto screen
2 participants
@EladBezalel @ThomasBurleson