Skip to content

Conversation

Youngemmy5956
Copy link

This PR fixes the issue where only the JavaScript of the first button is executed by processing each annotation individually.

[Bug]: JS of a button is not executed #19378

@Youngemmy5956
Copy link
Author

@nicolo-ribaudo can you please check the pull request I created for this

This PR fixes the issue where only the JavaScript of the first button is executed by processing each annotation individually.

[Bug]: JS of a button is not executed #19378

@nicolo-ribaudo
Copy link
Contributor

I don't have the knowledge to review the changes (you'll have to wait for the maintainers for that), but something that you also need to do is:

  • add a test, in test/integration/scripting_spec.mjs
  • squash the commits into a single one with a descriptive message

@g19i85g
Copy link

g19i85g commented Feb 5, 2025

Thanks @Youngemmy5956, I have tested it and can confirm that it works.

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

Successfully merging this pull request may close these issues.

4 participants