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

remove -webkit-transform: translateZ(0); because it breaks z-index properties #62

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mghignet
Copy link

Hi,

Great library!

I read that the -webkit-transform: translateZ(0); CSS property was used to enable hardware acceleration and make transitions more fluid. Is that why it is used on that library?

I found that when using the Window zoom option, the z-index property that you set to the window container is canceled and does not work. The result is that when you have "absolutely" positionned elements on the page with z-indexes, they appear over the zoom window, even if their z-index property is lower.
I fixed that by removing the -webkit-transform property.

@nubz
Copy link

nubz commented Jun 30, 2014

+1 I have issues when using this within an overlay - removing translateZ() works - currently using using lame call:

setTimeout(function () {
      $('div.zoomContainer').css('-webkit-transform', 'none');
}, 1000);

@aelamrani
Copy link

+1000

@mghignet
Copy link
Author

I have just rebased the branch so that it's mergeable again :)

@igorlino
Copy link

@mghignet Hi there, FYI, I have re-created the elevatezoom into EZPlus, an up-to-date project at:
https://github.com/igorlino/elevatezoom-plus
Although it has already many build changes, as of today it has nearly the same JS code as KyleMacey' fork (which had the most bug-fixes from all other elevatezoom forks. Thanks for your efforts)
Happy to receive PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants