Changes:
- Refactored articleForm.jsx by moving loadForm() before document.ready.then()
- Removed 'function' in .then() and modified to .then(() => {
- Removed unused function 'isUserSignedIn' in Onboarding.jsx
See: #2470
* Fix JavaScript Console errors in /Comments sub-page of any article
- Make use of `document.body.contains()` to check if the element is present before trying to Add or Remove Event listeners or assign attribute values to the same.
* Implement recommendations for changes
- Switch String.match() to Regex.test() in order to have consistent Boolean return.
- Switch if(document.body.contains(element)) to if(element) as no need to check if the element is in visible context, only whether present or not.
* Implement recommendations for changes
- Switch String.match() to Regex.test() in order to have consistent Boolean return for isIOSDevice().
- Switch if(document.body.contains(element)) to if(element) as no need to check if the element is in visible context, only whether present or not.
* Fix some eslint and a11y issues
* Add a11y fixed to snapshot
* Fix missing whitespace that cause test fail
* Fix code duplicate
* Config eslint to allows the label as a sibling
* Use 'error' instead of 2 in eslintrc
* Correct propTypes in SingleListing
* Add missing id prop for select
* Add space between two functions
* onKeyPress only register Enter
* Allow space key also to activate action
* Use a common function for three event handlers
* Add tag rules to article v2 editor
* Add bottom phrase to tag dropdown
* Remove console logs
* Remove empty space in css
* Remove commented code and clean up css
* Add type='button' for a11y
* Regenerate rubocop todo
* Fix outstanding problems
* Fix Style/FormatStringToken
* Final update to rubocop todo file
* Set RSpec/ExampleLength to max 10 and regenerate todo file
* fix use global form of use strict error
* fix errors involving functions being used before being defined
* fix undefined function and == errors
* fix redeclared variable name error
* use arrow functions to fix unnamed function warnings
* specify global function at top of file instead of disabling line in eslint, change meta variable name
- eslint configuration moved comments to a seperate line
in pre commit hook allowing the eslint-disable-line to
not work correctly.
- switched to eslint-disable-next-line to fix the same
