- 02 Aug, 2017 4 commits
-
-
Steel Wagstaff authored
Tiny fix to the test to account for URLs that may have had uppercase letters (the audio embed converts to lowercase to account for links to .MP3 files rather than .mp3)
-
Steel Wagstaff authored
Fixed tests for audio elements.
-
Steel Wagstaff authored
-
Robert Knight authored
Some API URLs are created by resolving relative URLs against the API root. For this to work properly, the root URL has to include a trailing slash. The omission of this in the documentation caused confusion for eLife devs.
-
- 01 Aug, 2017 4 commits
-
-
Robert Knight authored
The two most recent client releases failed to publish automatically because the latest tags had not been fetched prior to running `git describe` to check whether the git ref was a tag. I'm not sure exactly what changed and why this used to work before, but add an explicit `git fetch --tags` step. eg. See https://jenkins.hypothes.is/job/client/job/master/483/
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Remove the view switcher component
-
- 31 Jul, 2017 2 commits
-
-
Sean Hammond authored
Remove the (feature-flagged) view switcher component. We missed the chance to strike while the iron was hot on this one - neither Dawa or I have time anymore to work on further design iterations. I don't want to have dead code living behind a feature flag so I'm removing it. The code will still be in git if we ever want to revive it. Perhaps these tabs can be re-designed as part of a larger re-design of the sidebar in the future. I'll leave the GitHub issue about the usability issues with the existing selection tabs open: https://github.com/hypothesis/product-backlog/issues/327 For the record the main remaining issues with this were: 1. An undesirable visual popping happens when the view switcher loads on pages that have a lot of annotations. This also happens with the existing selection tabs but is arguably more visible with the view switcher. 2. When there are only two tabs (no orphans) the visual design of the tabs doesn't make it as immediately obvious which is the currently selected tab. The other tab may be mistaken for the selected one. The TODO list was here: https://github.com/hypothesis/product-backlog/issues/327#issuecomment-311073581 The pull requests for this view switcher were: https://github.com/hypothesis/client/pull/429 https://github.com/hypothesis/client/pull/481 https://github.com/hypothesis/client/pull/482 https://github.com/hypothesis/product-backlog/issues/327
-
Robert Knight authored
Adding ADR for ePub support
-
- 28 Jul, 2017 1 commit
-
-
Sean Roberts authored
-
- 27 Jul, 2017 2 commits
-
-
Sean Roberts authored
Adding onLayoutChange documentation
-
Sean Roberts authored
-
- 26 Jul, 2017 2 commits
-
-
Steel Wagstaff authored
More cleaning up of the media-embedder tests
-
Steel Wagstaff authored
Cleaned up media-embedder tests
-
- 25 Jul, 2017 7 commits
-
-
Steel Wagstaff authored
Added media-embedder tests for .mp3, .ogg, .wav files
-
Steel Wagstaff authored
Turns links to web-published .mp3, .ogg, and .wav files into HTML5 audio elements
-
Steel Wagstaff authored
-
Robert Knight authored
-
Robert Knight authored
-
Sean Roberts authored
Fix IE 11 regression in documents without a `<base>` element.
-
Robert Knight authored
`Document#uri` and `Document#getMetadata` returned the base URI rather than the document's real URL as the document location if the protocol of the base URI and true document location were different. The reason for this was to support the use case where the document is an iframe loaded from a `blob:` URL, which Readium does for example. This broke in IE 11 if the document did not have a `<base>` element because the fallback `Node.baseURI` and `Document.documentURI` properties used by `document-base-uri` do not exist in that browser. This commit re-works the behavior of `_getDocumentHref` to be more conservative and return the base URI only if a) the document location does not have a whitelisted scheme and b) the base URI is defined and does have a whitelisted scheme. Additionally, the tests have been re-worked to test the public `uri` method instead of the private `_getDocumentHref` method and use test seams instead of proxyquire.
-
- 21 Jul, 2017 3 commits
-
-
Steel Wagstaff authored
-
Steel Wagstaff authored
Code clean up (removed H5P embed test)
-
Steel Wagstaff authored
Added support for mp3 -> html5 audio and a primitive h5p embedder.
-
- 20 Jul, 2017 2 commits
-
-
Robert Knight authored
-
Robert Knight authored
-
- 19 Jul, 2017 2 commits
-
-
Robert Knight authored
Use a URN derived from DC meta tags for creating equivalent doc links
-
Juan Corona authored
“dc.relation.ispartof” is now used instead of “dc.source” The goal of this is to identify a resource that’s part of another resource, therefore the more specific term fits better.
-
- 18 Jul, 2017 1 commit
-
-
Juan Corona authored
Use a URN derived from DublinCore meta tags for creating equivalent links across documents that provide ‘dc:source’ and ‘dc:identifier’ tags This link is now the documentIdentifier for non-PDF cases for the document metadata. This metadata is shared with the sidebar, making this link appear in the list of links for the Search API calls
-
- 17 Jul, 2017 4 commits
-
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Adding support for onLayoutChange notifier configuration
-
Robert Knight authored
- Add a doc comment to the _notifyOfLayoutChange function to clarify the purpose of the optional param. - Add a comment inside the function to describe the high-level structure of the sidebar and how expanding/collapsing it is achieved. - Correct a typo with "frameVisbileWidth"
-
- 14 Jul, 2017 6 commits
-
-
Sean Roberts authored
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Capture the client url to be used in subframes
-
Robert Knight authored
When the client served by the browser extension injects itself into child iframes on the current page, it needs to include the "assetRoot" config setting in order for the client in the iframes to be loaded from the extension rather than the default CDN URL.
-
Sean Roberts authored
-