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

Added support for image rotation and enhanced some features #9

Open
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

jmalarcon
Copy link
Contributor

@jmalarcon jmalarcon commented Feb 15, 2020

Hi Samil:

I've enhanced the plugin with several features:

  • [NEW] Image rotation support, using transforms for rotation and pan position. This removes support for IE versions before 10. (IE10+ supported with jQuery 1.x).
  • [Enhancement] New icons. The original zoom in, zoom out and close icons were not easily spotted with very dark or very light big images (not enough contrast). I've added a subtle glow effect and changed the color slightly, and they are now very easily spotted under most circumstances.
  • [Enhancement] Added support for an animated loading indicator that is shown while the big image loads. Yo had that in mind but was not implemented.
  • [Bug] Fixed a bug: When a new image was loaded the previous zoomed one was shown while the new loaded. Now the previous one is unloaded before the new starts to load and the progress indicator is shown.

Hope you like at and merge my changes with yours.

Best regards

jmalarcon and others added 30 commits December 18, 2018 17:06
Updated npm dependencies
- Allow zooming elements with several images inside
2. Extend loading duration for UI improvements
- Added the option to call pan on dynamically added images
- Added the option to delay displaying the images n an specific amount of milliseconds
Dynamically add pan to new loaded images
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.

2 participants