- 09 Mar, 2016 2 commits
-
-
Robert Knight authored
-
Robert Knight authored
It accepts a list of CoffeeScript file paths on stdin and for each <path>.coffee file in the input, it writes a <path>.js output file containing the converted, reformatted source. The result is ES2015 source so will require some fixups as long as we're using ES5 in the main app codebase. The current transformation process involves: 1. Performing an initial CoffeeScript -> JS conversion with 'decaffeinate' 2. Applying some fixups for common issues, currently done with dumb string modifications. In future I might look at writing a Babel transformer. 3. Parsing the source with Babylon to check that it is valid ES2015 If not, the user is prompted to go and simplify the CoffeeScript so that decaffeinate can process it correctly. 4. Reformatting the source with typescript-formatter, which takes care of indentation and high-level formatting. 5. Running the source through JSCS' fix mode which deals with smaller details.
-
- 08 Mar, 2016 2 commits
-
-
Nick Stenning authored
Fix installation of URL polyfill with js-polyfills v0.1.16
-
Robert Knight authored
js-polyfills v0.1.15 used to install the polyfill on `this.URL`, where `this` was set to module.exports in the context of a Browserify bundle. Consequently the result was available as `require('js-polyfills/url').URL`. As of v0.1.16 however, it installs the polyfill on `self`, which is the Window object in the context of the browser and so `require('js-polyfills/url').URL` is undefined. Fixes #3064
-
- 07 Mar, 2016 5 commits
-
-
Nick Stenning authored
Add extra context to sentry logging
-
Robert Knight authored
Merge pull request #3057 from hypothesis/sheetaluk/280-when-a-user-replies-focus-on-and-expand-that-reply Sheetaluk/280 when a user replies focus on and expand that reply
-
Sheetal Umesh Kumar authored
added missing new line. don't attempt toggleCollapsed if no parent available adding a test and a comment simplifying assert
-
Nick Stenning authored
Convert {app, app-controller, widget-controller}.coffee to JS
-
Nick Stenning authored
Adds an issue template
-
- 05 Mar, 2016 2 commits
-
-
Robert Knight authored
- Extract the logic computing a numeric key for sorting annotations by location into a separate module to make testing easier and add tests.
-
Robert Knight authored
In preparation for upcoming changes as part of direct linking, convert the entry point and top-level controllers for the app to JS. * Use a CommonJS require to bring in the 'vendor/jwz' dependency.
-
- 04 Mar, 2016 8 commits
-
-
Robert Knight authored
Travis Sentry sourcemaps
-
Robert Knight authored
Throwing an exception in the context of a Promise catch handler inside through.obj() just results in the exception being silently swallowed. Pass it to the through2 callback so that Gulp exits with the correct status.
-
Nick Stenning authored
This commit reworks scripts/gulp/upload-to-sentry.js to allow for uploading to multiple Sentry projects at once. This allow us to build the assets once, and upload them to multiple Sentry projects within the same gulp invocation.
-
Nick Stenning authored
-
Christof Dorner authored
Run search results through AnnotationJSONPresenter
-
Nick Stenning authored
Fix JS error checking via Hound
-
Robert Knight authored
Since enabling JSCS, Hound CI reviews for PRs which contain JS code changes have been failing to complete. ThoughtBot have advised us that the JSCS checker is in beta and that we should simply disable it. They noted that ESLint has received a better response from other testers so we might want to look at that instead.
-
Robert Knight authored
- Use the legacy 'esnext' option only instead of both 'esnext' and 'esversion' for compatibility with JSHint v2.8.x on Hound and developers who have the current stable version installed locally. - Remove the 'maxErrors' option from the JSCS config, as this prevents use of the '-x' option to autofix files which is a little annoying.
-
- 03 Mar, 2016 2 commits
-
-
Nick Stenning authored
Fix ubuntu dev install docs
-
Sean Hammond authored
Focus input field after creating a new annotation
-
- 02 Mar, 2016 2 commits
-
-
Robert Knight authored
The previous method of scrolling a newly created annotation into view in the sidebar resulted in the input field for the new annotation losing focus. Note that only top level annotations have an 'id' attribute set on the corresponding HTML element. It might make sense to change this in future although it would add an extra binding. For now I've just noted this.
-
Robert Knight authored
When a new annotation card was added, the sidebar tried to focus it but this failed because the sidebar iframe itself did not have focus. When a new annotation is created, first focus the sidebar in the Annotator host and then let the sidebar focus the input field. * Remove unnecessary use of the $timeout service which triggers a root scope digest.
-
- 01 Mar, 2016 8 commits
-
-
Robert Knight authored
Split Travis build up using matrix
-
Nick Stenning authored
This allows `gulp build` to run without devDependencies installed.
-
Nick Stenning authored
Implement tools to enable Chrome extension sourcemaps
-
Robert Knight authored
Hound CI is currently using JSHint v2.8.0 which uses 'es3', 'es5' and 'esnext' config options instead of the 'esversion' option in v2.9.0 and later.
-
Robert Knight authored
When the H client is served from a local URL (eg. chrome-extension://ID/scripts/foo.bundle.js), the original source URLs and source maps are not accessible to Sentry. Enabling source maps for extensions is done in three steps, the first two of which are implemented in this commit: 1. Changes to the crash reporting to transform local URLs into filenames when reporting exceptions. This results in URLs for source files in Sentry reports which are independent of where the files are served from (eg. chrome-extension://ID, moz-extension://ID...) 2. A Gulp task that uploads JS bundles and their associated sourcemaps to Sentry using the Sentry API. The assets are uploaded to Sentry using just the filename as the URL, so the source files and source maps are correctly matched up with the transformed URLs produced by the changes in (1). 3. Configuring CI builds to run the Gulp task from step (2). This commit makes use of ES2015 template strings in upload-to-sentry.js, so the JSHint file has been adjusted accordingly.
-
Nick Stenning authored
Remove undocumented /api/annotations endpoint
-
Nick Stenning authored
Convert fuzzy timestamp formatting functions to JS
-
Robert Knight authored
In preparation for changing the formatting of breakpoints, convert this module to a plain CJS module in JS.
-
- 29 Feb, 2016 6 commits
-
-
Robert Knight authored
Clean up application config/settings
-
Nick Stenning authored
This way it's possible to run `gulp build` without installing the development dependencies.
-
Robert Knight authored
Standardise and improve log output for all servers
-
Robert Knight authored
Upgrade to Pyramid 1.6
-
Nick Stenning authored
Add a test and fix a parameter for queries
-
Nick Stenning authored
Refactor and add tests for annotation editor toolbar commands
-
- 26 Feb, 2016 3 commits
-
-
Sean Hammond authored
Move API token model to app
-
Robert Knight authored
Previously all modified lines in a block were replaced in one call to replaceText() when applying block formatting, which lost the selection. This rewrites the command to transform each line separately which preserves the selection.
-
Robert Knight authored
* Separate the logic for transforming the input field from the logic for actually reading the state of the input field and applying changes. This enables testing of the commands without instantiating the component. Toolbar commands are functions which take the current state of the input field and return the updated state. * Add tests for the individual commands and use of the commands in the markdown editor.
-