danielweck
commented
6 years ago As per last week's conference call:
1) disable all plugins (or to be precise: "do not enable" any plugin) via the plugins/plugin.cson configuration file in readium-shared-js ( https://github.com/readium/readium-shared-js/blob/develop/plugins/plugins.cson ), so that the build-output folder (which is used as-is by the ReadiumSDK launcher apps) is as "vanilla" as possible ( https://github.com/readium/readium-shared-js/tree/develop/build-output ).
2) Enable the Hypothesis plugin (or any other desired plugin) in the develop branch via TravisCI during the Continuous Integration build that is automatically triggered at each code push. As a result, the automated cloud reader deployment at Firebase and Surge continues to provide the Hypothesis annotations tool ( http://readium.firebaseapp.com and https://readium.surge.sh ), as well as the cloud-reader[-lite].zip archives (ready-made prerelease builds) which are conveniently offered to downloaders who do not want / cannot compile the app themselves ( https://github.com/readium/readium-js-viewer/releases/tag/0.28.0-alpha ).
3) The manual builds created from the master branch (tagged releases) are not affected, they do not contain any plugins ( https://github.com/readium/readium-js-viewer/releases/latest ), and the "stable" cloud reader app deployed at Surge does not offer the Hypothesis plugin ( https://readium-master.surge.sh ).
We are currently testing this strategy live: https://travis-ci.org/readium/readium-js-viewer/builds/294962188 (waiting for the build to finish...)
This issue is a Discussion
For quite a while Readium has supported a plugin framework centered on JavaScript. Although present, it has not been first-class citizen. One plugin, the Highlighter, was written early on, but there is no associated "bookmarking or annotation" plugin using it. There is some usage (we think) of the highlight plugin and the framework out in the wild, but we have little solid evidence of how or how much it used.
Meanwhile, EvidentPoint worked with hypothes.is to enable their bookmarking/annotation support within the CloudReader. It is based on a wholly different framework for highlighting, which is not compatible with the existing framework. So the question has arisen, how should Readium support Hypothes.is and/or the existing Highlighter plugin (which is disabled by default)?
At present, in the develop branch, in preparation for release 0.28 (which is intended to support Hypothes.is) the Hypothes.is plugin is enabled by default and the Readium Highlighter is disabled. At first, this seemed OK, we provide instructions on how to disable the Hypothes.is plugin so anyone deploying the CloudReader who has their own highlighter and/or doesn't want the Hypothes.is support can do so. Unfortunately, it is a little more complex than that. As it stands the Hypothes.is support, as part of readium-shared-js, will be built and integrated into not only the CloudReader but also the Chrome app and the native SDK apps (the Launchers). It won’t work in the Chrome app or the Launchers but the code is there and even the UI.
This is clearly unacceptable, but how should it work? After some discussion, it seems that the requirements should be:
The intent of this issue (discussion) is to elucidate how the above requirements can be met for the 0.28 release.
Related issue(s) and/or pull request(s)
657
Test file(s)
None
Product
All
Additional information
None?