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

feat: new disableTransform prop to use top/left instead of translate #138

Merged
merged 6 commits into from
Jul 28, 2023

Conversation

AousAnwar
Copy link
Contributor

A small change for a particular use case.

The css Transform property create a new scope inside the browser, so the item with fixed position is positioned relatively to the nearest parent with the transform property (.vue-recycle-scroller__item-view) and not the browser itself.

Here I'm adding a "modal" property the the item object, when set to true, it replace "transform (x or y)" with "left or top" depending on the "direction" also unset the css "will-change" property.

Typical use case : zoom in modal for images or video display

…xed position item

A small change for a particular use case.

The css Transform property create a new scope inside the browser, so the item with fixed position is positioned relatively to the nearest parent with the transform property (.vue-recycle-scroller__item-view) and not the browser itself.

Here I'm adding a "modal" property the the item object, when set to true, it replace "transform (x or y)"  with "left or top" depending on the "direction" also unset the css "will-change" property.

Typical use case : zoom in modal for images or video display
@kilkujadek kilkujadek mentioned this pull request Jan 6, 2020
ploissken pushed a commit to ploissken/vue-virtual-scroller that referenced this pull request Jan 7, 2020
@crow7m
Copy link

crow7m commented Feb 6, 2020

aby updates on this pull request, might be useful for dynamic scroller as well

@bobaoapae
Copy link

Hello, can this be merged? I have this issue

@Akryum
Copy link
Owner

Akryum commented Oct 14, 2022

I'm not sure we want to change this as it might degrade performance.

@netlify
Copy link

netlify bot commented Jul 28, 2023

Deploy Preview for vue-virtual-scroller-demo failed.

Name Link
🔨 Latest commit ed426ae
🔍 Latest deploy log https://app.netlify.com/sites/vue-virtual-scroller-demo/deploys/64c3bf7da06d0900089dc900

@Akryum Akryum changed the title Modal : Change the "transformX|Y" css property to "top or left" for a fixed position item feat: new disableTransform prop to use top/left instead of translate Jul 28, 2023
@Akryum Akryum merged commit 547cf52 into Akryum:master Jul 28, 2023
0 of 6 checks passed
@mustafaBagwala
Copy link

@Akryum Which version of the package contains this fix please ?

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.

None yet

6 participants