forked from share-extras/media-viewers
-
Notifications
You must be signed in to change notification settings - Fork 0
Last update #1
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
Merged
Merged
Last update #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added latest pdf.js lib a081c2d. This version of pdf.js supports range request. This is however disabled default in PdfJs.js. If you want to experiment with this feature change to PDFJS.disableRange = false. Initial testing gives that there are no speed benefits.
Also handles the error discovered in pdf.js while investigating #8
…nt 4.x versions of Alfresco.
…d by Alan Davis in ALF-19056 - Tested locally, verified this fixes global anything-to-anything transformation problems
…fig - otherwise this will break compatibility with 4.0/4.1
…ties to prevent possible name clashes
…prevented the add-on from working with Alfresco 4.2.c - Tested operation on 4.2.d, verified that startup still completes and video transforms work as expected
separate beans - Allows the transformation parameters to be controlled much more easily - MP4 transformation options are now much simplified, reducing the risk of specific flags not being supported
The core pdf.js has split workers to a separate file (pdf.worker.js). In theory we would not need this file to be loaded via web-preview.get.head.ftl but in order to have this file checksummed and to find out the checksum name we need to do this. This means it is loaded twice. The second time will however be from cache. Probably means parsed twice as well. I've not seen any noticeable effects on loading performance due to this.
- Check max/min page number supplied in hash. Instead of throwing error resets to max available pages or page 1. This makes bookmarked url always work even if the number of pages in document change. It may not link to intended page, but does not error out, and we can hope the last page is a good enough target. - Introduced new internal attribute this.disabledPageLinking that default to true unless page is document-details. In doing so the page hast property will be ignored and paging for document list will still work. This commit does not remove the linking button on pages other than document-details. This will be done as part of #12.
- Add priority=50 to FFmpeg-based transform rules - fixes warnings on startup - Remove audio/webm transform since this mimetype does not seem to exist in the repository
- Toolbar config is specified in model.toolbarItems, which can be modified as needed by extensions
…only enabled on the document details page (ref. #27)
…as otherwise we cannot easily get back to where we were when we resume the presentation
…er viewers via XHR - Use thumbnailModification option on WebPreview instance if available - Detect the quickshare page and if we still do not know what has been pre-generated then just assume blindly that we can generate as needed (fixes #29)
- Added missing translations - Corrected encodings
…epository.url which can be overridden for deploying artifacts
…dules, but now in combination with jsonUtils.toObject()
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.