Skip to content

Module upgrades and bug fixes #85

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

saipavan10-git
Copy link

@saipavan10-git saipavan10-git commented Jul 9, 2025

This PR changes involve upgrading the framework version in config.json and removing the use of JavaScript eval() from the imagemapster.js file, with documentation added to the developer README about the procedure.

- Register callback to afterRender to prevent duplicate renders
- closes ctsit#80
- Check for the initialized and unbind maps generated later using a cleanup function
- Remove eval from the code and minify the js
- Update the dev readme with the steps
@saipavan10-git saipavan10-git marked this pull request as ready for review July 10, 2025 19:00
@saipavan10-git saipavan10-git requested a review from pbchase July 10, 2025 19:07
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.

1 participant