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

Update jquery-ui and jszip modules to advance firefox compatibility efforts #46

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

Conversation

bomberm
Copy link

@bomberm bomberm commented Jun 6, 2019

  • The assistant requires two JavaScript Libraries that are considered unsafe by Firefox.

    • Stuk-jszip needed to be moved to the current version from https://github.com/Stuk/jszip. I have tested using jszip master and the RecorderAssistant appears to run well with this update. The manifest.json reflects this new version as well.
    • jquery-ui used a depricated version of jquery. The jquery-ui module is compatible with newer jquery versions but is not packaged with one, so I have updated the internal jquery version.
  • I have created a guide (labelled Firefox Add-On generation) that expands upon the windowsclub guide to fill in the gap of work needed to generate a firefox xpi so that future work is not starting from zero. This has steps not only for the xpi generation, but also the Firefox Add-on signing process.

This pull request is for effort to address issue #11, but does not fully resolve the issue.

The tools recommended within issue #11 are not enough to convert the GenderMagRecorderAssistant. While I have been able to create a test .xpi, dependencies on chrome specific methods meant the assistant was non-functional. This pull request is an effort to bring GenderMagRecordersAssistant closer to firefox compatibility.

roseg31 pushed a commit that referenced this pull request Aug 24, 2020
Ensures previously saved subgoals are not overwritten when the user clicks on them before saving the current subgoal. Calling drawSubgoal with a different id is restricted to after the current subgoal has been saved.
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