- 27 Feb, 2017 14 commits
-
-
Robert Knight authored
This is now just getting distracting amidst the more important information that gets logged during a build.
-
Robert Knight authored
Print the sidebar app URL and asset root URL that were baked into the client after the first build. This is useful to verify that the build was run with the expected environment.
-
Robert Knight authored
To facilitate testing the client on devices that are not the same as the one the package server and Hypothesis dev server are running on, it is useful to be able to set the hostname for URLs that point to the package content server that `gulp watch` runs. This commit adds support for setting this via a `PACKAGE_SERVER_HOSTNAME` env var.
-
Robert Knight authored
When building the client in development mode, make the client load assets from the package content server by default. Also if `H_SERVICE_URL` is set, bake that service's /app.html endpoint into the boot script.
-
Robert Knight authored
The live reload server's example page was modified to use a script tag for config settings because the boot script currently only reads that config source and does not use `window.hypothesisConfig()`. In order to use the sidebar app from the local service, the service needs to be configured to serve the local client by setting the 'CLIENT_URL' env var before running the devserver and in the client, the 'H_SERVICE_URL' env var needs to be pointed at the devserver host.
-
Robert Knight authored
Make the package content server mirror unpkg's behavior in that: - The entry point for a package is located at `/{packageName}@{version}` - Requests without a package version specified redirect to the corresponding path for the latest package version
-
Robert Knight authored
Add an express server which serve's the package's contents in a manner similar to unpkg: - The '/' route serves the package's entry point script - Other routes serve files from the package This can be used together with the CLIENT_URL setting in the Hypothesis service to configure the service to serve the local development version of the client instead of the production version.
-
Robert Knight authored
Fix PDF quote anchoring
-
-
Robert Knight authored
The previous syntax ended up executing: docker run -d hypothesis s3://cdn.hypothes.is nickstenning/s3-npm-publish So `hypothesis` was interpreted as the Docker image name instead of an argument. Passing the args as the second ('command') argument to `Image.withRun` results in the correct command being executed: docker run -d nickstenning/s3-npm-publish hypothesis s3://cdn.hypothes.is
-
Robert Knight authored
-
Robert Knight authored
Upload the client to the S3 bucket behind cdn.hypothes.is after publishing to npm. The service will then redirect requests for /embed.js to cdn.hypothes.is/hypothesis
-
Robert Knight authored
-
Robert Knight authored
We decided to publish the client's assets to a Hypothesis-specific domain because publishers (eg. https://internethealthreport.org) may want to use CSP to restrict where assets are loaded from and they won't want to whitelist the whole of npm. This also gives us better control over caching headers used on assets.
-
- 24 Feb, 2017 4 commits
-
-
Robert Knight authored
Improves performance and includes a number of bug fixes. See https://github.com/showdownjs/showdown/blob/master/CHANGELOG.md for changes. Several tests required small changes because the rendering of `para1\n\npara2` changed from: `<p>para1</p>\n\n<p>para2</p>` To the visually equivalent: `<p>para1</p>\n<p>para2</p>`
-
Robert Knight authored
This test fails without the `toPositionAnchor` fix in the previous commit.
-
Robert Knight authored
This method used to exist in the `dom-anchor-text-quote` library but got lost when the class was moved to a wrapper in `types.coffee` after the upgrade to dom-anchor-text-quote v3.x. This method is used by PDF anchoring and its absence caused quote anchoring to throw an exception. The majority of PDF annotations still continued to anchor because the position anchor still worked. This commit also adds some basic documentation to the `types` module and some basic API tests.
-
Robert Knight authored
Make this file easier to understand for the next person who tries to read it.
-
- 22 Feb, 2017 2 commits
-
-
Sheetal Umesh Kumar authored
-
Robert Knight authored
The release notes were missing on the GitHub Releases page for the client for several recent releases. This happened because of incorrect heading levels used for subsections within a release in the CHANGELOG file.
-
- 21 Feb, 2017 8 commits
-
-
Robert Knight authored
Don't treat annotation quotes (TextQuoteSelector.exact) as HTML
-
Robert Knight authored
-
Sean Roberts authored
-
Sean Roberts authored
-
Sean Roberts authored
-
Sean Roberts authored
-
Sean Roberts authored
-
Nick Stenning authored
The annotation quote, as plucked from the TextQuoteSelector, is not HTML, and so shouldn't be rendered as such. Fixes hypothesis/h#2896.
-
- 20 Feb, 2017 4 commits
-
-
Sean Roberts authored
-
Sean Roberts authored
-
Sean Roberts authored
Add invisible threads list to allow render time saving for out of vie…
-
Sean Roberts authored
-
- 17 Feb, 2017 6 commits
-
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
-
Nick Stenning authored
Add a new entry point script for the client application
-
Robert Knight authored
Remove account settings and logout link from login control for 3rd pa…
-
Sheetal Umesh Kumar authored
For 3rd party users: Remove the 'Account settings' link from the account menu Remove the logout link from the account menu Related PR: https://github.com/hypothesis/client/pull/228 Fixes: hypothesis/product-backlog#141
-
- 16 Feb, 2017 2 commits
-
-
Robert Knight authored
Disable the link to activity pages from user profile, if the user is …
-
Robert Knight authored
correcting CC0 terminology
-