- 19 Jul, 2016 4 commits
-
-
Robert Knight authoredSet the `ng-csp` attribute on the document body on app startup before Angular is loaded to avoid triggering warnings when app.html is served with Content Security Policy headers. This used to be done via an `ng-csp` attribute on the <body> tag in the app.html file served by the `h` service but this moves it to client code to avoid the service needing to know this implementation detail of the client app.
-
Robert Knight authored
This was needed when this part of the app template lived in the `h` repository. It is not needed any more.
-
Robert Knight authored
Move the template which defines the structure of the top-level of the UI from `app.html.jinja2` in the hypothesis/h repo into the client. This means that the H service only needs to serve an app.html page containing a `<hypothesis-app></hypothesis-app>` placeholder in the body which the app will then load into.
-
Robert Knight authored
Remove SASS variables and mixins which are not used by the client
-
- 15 Jul, 2016 16 commits
-
-
Nick Stenning authored
ESLint - Enforce indentation, quote style and `this` style consistency
-
Robert Knight authored
Since this module was imported with minor changes from substack/frame-rpc, disable ESLint to avoid adding noise to the Git history for this file.
-
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Use 2-space indents everywhere and fix up non-conforming code with ESLint's --fix tool.
-
Robert Knight authored
Only allow use of `this` or `self` to refer to the 'this' context object in a method.
-
Robert Knight authored
Add an ESLint rule to require this and fix up non-conforming code using ESLint's --fix tool.
-
Robert Knight authored
Refactor the streamer into a stand alone class
-
Alice Wyan authored
-
Alice Wyan authored
-
Sheetal Umesh Kumar authored
-
Robert Knight authored
The group invite icon is only used in the H web service
-
Robert Knight authored
* The corejs polyfills are not needed in the Gulpfile since we switched to a more recent version of Node * The Object.assign() polyfill does not need to be included in 'raven' or 'settings' because it is already included by the polyfills.js script
-
Robert Knight authored
This was left over from when the Hypothesis client was extracted out from the hypothesis/h repository, where the extension code currently remains.
-
Robert Knight authored
Enforce that all paths through a function either return a value or do not return a value. This cause several cases of spurious returns left in by CoffeeScript -> JS conversion and return statements with missing values.
-
- 14 Jul, 2016 13 commits
-
-
Sheetal Umesh Kumar authored
Fix 'time' tests in timezones behind UTC
-
Robert Knight authored
Several test fixtures have expectations that the "current date", where "current" is reset to the Unix epoch at the start of the tests, is 1st Jan 1970. This is true in timezones equal to or ahead of UTC but not the case for timezones behind UTC. Resolve the issue by just advancing the fake clock forwards to midnight on 01/01/1970 if the timezone is behind UTC at the start of the tests.
-
Sheetal Umesh Kumar authored
-
Alice Wyan authored
-
Sheetal Umesh Kumar authored
Annotation / Notes tabs copy update and style tweaks
-
Sheetal Umesh Kumar authored
Update UI state after search API request completes with no results
-
Sheetal Umesh Kumar authored
Remove sticky tab bar behavior
-
Robert Knight authored
When a search API request returns results, the returned annotations are added to the app state and this transition triggers a digest cycle. When there are no results however, this was not happening and the value of the 'isLoading()' function changed but no digest cycle was triggered to pick this up and reflect it in the UI. Since SearchClient's 'end' event is emitted asynchronously via a Promise callback, $scope.$apply() or $evalAsync() must be used to ensure that change detection runs. The real solution will be to store all the information required to determine the loading/not-loading state in the Redux store, so that any transition automatically triggers a digest cycle. This requires some more extensive refactoring however.
-
Robert Knight authored
The sticky tab bar behavior has some UI issues that need to be resolved before we could ship it. For example, when switching between tabs the scroll position needs to be reset to the top. For the time being, this commit removes the behavior.
-
Robert Knight authored
Following feedback and design review, change the label from 'Notes' to 'Page Notes' to better distinguish notes from annotations and make the empty state text for the Annotations tab more helpful by explaining to users how to create a new annotation.
-
Robert Knight authored
Shift the 'No annotations in this group' message down slightly to prevent the border from being clipped. Also prevent text selection in the tabs.
-
Nick Stenning authored
ESLint - Enable 'Best practices' rules
-
Robert Knight authored
Usage of Hound has been replaced by a Travis task that runs `npm run lint`
-
- 13 Jul, 2016 7 commits
-
-
Robert Knight authored
Enable additional rules from the 'Best Practices' list on ESLint's Rules page.
-
Robert Knight authored
Use `.delete` instead of `['delete']` etc. since all browsers we target support ES5.
-
Nick Stenning authored
Add ESLint configuration and use it to replace JSCS / JSHint
-
Robert Knight authored
-
Robert Knight authored
These have been replaced by ESLint
-
Robert Knight authored
For stylistic consistency, require semicolons rather than relying on ASI.
-
Robert Knight authored
Checking for return values in Array callbacks found several incorrect uses of Array.map() where Array.forEach() should have been used.
-