- 15 Jul, 2016 11 commits
-
-
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 16 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.
-
Robert Knight authored
This will aid migration to let/const in future
-
Robert Knight authored
This will ease migration to let/const in future.
-
Robert Knight authored
-
Robert Knight authored
- Remove unused variables. Note that ESLint is smarter than JSHint in that it realizes that vars which are assigned but never read are unused. - Add missing `angular` require statements. - Add ESLint suppressions where necessary
-
Robert Knight authored
Add a basic ESLint configuration which uses ESLint' recommended settings to check for basic errors such as undefined vars and unused variables.
-
Nick Stenning authored
-
Nick Stenning authored
-
Nick Stenning authored
-
Sheetal Umesh Kumar authored
Improve tab bar input handling and accessibility
-