| Age | Commit message (Collapse) | Author |
|
The loadstart even is only triggered once to indicate the progress has
begun (initialize the progress bar). Essentially, this is dead code for
two reasons:
1. It never gets emitted because initProgressBar() gets emitted first.
2. No JavaScript file in pomf core uses the non-standard uploadstart
event.
|
|
Using anonymous functions with event handlers is painfully slow
according to benchmarks. Anonymous functions are up to 99% slower.
It is also illegal to use anonymous functions in this way according to
Airbnb ES5 style guide.
Fixes some ESLint func-names errors using Airbnb ES5 config.
Reference: https://jsperf.com/event-handler-vs-anonymous-function/3
|
|
Until the code has been converted to ES6 in it's entirety, let's stick
to Airbnb's ES5 style guide so that ESLint won't complain.
This reverts commit e6f354a.
|
|
Fixes JSCS safeContextKeyword warnings using Airbnb preset.
|
|
Fixes JSCS disallowQuotedKeysInObjects warnings using Airbnb preset.
|
|
This adds commas to cheesesteak.js.
Something to note, the Airbnb ES5 style guide opposes a trailing comma.
The latest ES6 edition endorses adding a trailing comma, explaining it
leads to cleaner Git diffs. Endorse the ES6 standard, that's also the
Airbnb preset in JSCS.
Fixes JSCS requireTrailingComma warnings using Airbnb preset.
|
|
Airbnb says to use /** ... */ for multiline comments and // for single
line comments.
Fixes JSCS requirePaddingNewLinesBeforeLineComments and
validateIndentation warnings using Airbnb preset.
|
|
Fixes JSCS validateQuoteMarks warnings using Airbnb preset.
|
|
Fixes JSCS disallowTrailingWhitespace warnings using Airbnb preset.
|
|
Fixes JSCS requirePaddingNewLinesAfterBlocks warnings using Airbnb
preset.
|
|
Fixes ESLint warnings about no-irregular-whitespace.
|
|
Reduces confusion and improves readability, despite not having yet
agreed on which JavaScript style guide to use.
|
|
|
|
This code doesn't seem to be in use nor important. There is no good
reason why this was left in here. Because it's not been in use for so
long and nobody has an issue with it, let's just remove it.
|
|
|
|
Adds curly braces and semicolons. Also change indentation to two spaces.
|
|
|
|
|
|
We're gonna use this shortly for the data field
|
|
|