-
Notifications
You must be signed in to change notification settings - Fork 32
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
5703a97
commit fa1ca9d
Showing
1 changed file
with
8 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi guy
Can i ask you question
How i can use it with multiple images like website hellomonday.com ?
When i used with multipe image it will broken
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to program that bit in. right now its only full screen, but im working on a new version in javascript that will be able to do this
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, i was just update your source and it can work with multiple images :). Thanks for your code
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main problem with multiple image performance can be found here
The default for element when its not specified should be
@settings.container
, and notwindow
. What did you change to get it to work? I'm adding commits daily to the dev branch, if you want to see progress there.fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have just use container by main element for each images but event hover of you using mouse window, i fixed it to move on element, it work.
Please see image below
http://prntscr.com/8j27hy
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, cool. Eventually, I want to have 4 different modes for the mouse movement.
Full - tracks the entire page
Isolated - tracks within the container
Magnetic - tracks within a certain distance from the container
Cross - Closest to the hellomonday profile, where if an element is on the same x/y plane, it will activate the effect.
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, When you update new code, please notify or comment in here, i will check and test, i like your code, but your effect not smooth like hellomonday.com (I was just read code it, and i see they calculate matrix different you )
Thanks.
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, hellsan631
I just test your new code, but it not good if you use >30 images
My web
http://prntscr.com/8mtbje
Your demo 5
http://prntscr.com/8mtbrq
I will update my changes to git
Please test with 30 images and remove overflow hidden in body (css)
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would you be able to open a new issue for this?
fa1ca9d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just create issue in here
#15