4minitz/4minitz

View on GitHub

Showing 435 of 452 total issues

Strings must use singlequote.
Open

        console.log("Error: "+error);
Severity: Minor
Found in private/importMeetingSeries.js by eslint

enforce the consistent use of either backticks, double, or single quotes (quotes)

JavaScript allows you to define strings in one of three ways: double quotes, single quotes, and backticks (as of ECMAScript 6). For example:

/*eslint-env es6*/

var double = "double";
var single = 'single';
var backtick = `backtick`;    // ES6 only

Each of these lines creates a string and, in some cases, can be used interchangeably. The choice of how to define strings in a codebase is a stylistic one outside of template literals (which allow embedded of expressions to be interpreted).

Many codebases require strings to be defined in a consistent manner.

Rule Details

This rule enforces the consistent use of either backticks, double, or single quotes.

Options

This rule has two options, a string option and an object option.

String option:

  • "double" (default) requires the use of double quotes wherever possible
  • "single" requires the use of single quotes wherever possible
  • "backtick" requires the use of backticks wherever possible

Object option:

  • "avoidEscape": true allows strings to use single-quotes or double-quotes so long as the string contains a quote that would have to be escaped otherwise
  • "allowTemplateLiterals": true allows strings to use backticks

Deprecated: The object property avoid-escape is deprecated; please use the object property avoidEscape instead.

double

Examples of incorrect code for this rule with the default "double" option:

/*eslint quotes: ["error", "double"]*/

var single = 'single';
var unescaped = 'a string containing "double" quotes';

Examples of correct code for this rule with the default "double" option:

/*eslint quotes: ["error", "double"]*/
/*eslint-env es6*/

var double = "double";
var backtick = `back\ntick`;  // backticks are allowed due to newline
var backtick = tag`backtick`; // backticks are allowed due to tag

single

Examples of incorrect code for this rule with the "single" option:

/*eslint quotes: ["error", "single"]*/

var double = "double";
var unescaped = "a string containing 'single' quotes";

Examples of correct code for this rule with the "single" option:

/*eslint quotes: ["error", "single"]*/
/*eslint-env es6*/

var single = 'single';
var backtick = `back${x}tick`; // backticks are allowed due to substitution

backticks

Examples of incorrect code for this rule with the "backtick" option:

/*eslint quotes: ["error", "backtick"]*/

var single = 'single';
var double = "double";
var unescaped = 'a string containing `backticks`';

Examples of correct code for this rule with the "backtick" option:

/*eslint quotes: ["error", "backtick"]*/
/*eslint-env es6*/

var backtick = `backtick`;

avoidEscape

Examples of additional correct code for this rule with the "double", { "avoidEscape": true } options:

/*eslint quotes: ["error", "double", { "avoidEscape": true }]*/

var single = 'a string containing "double" quotes';

Examples of additional correct code for this rule with the "single", { "avoidEscape": true } options:

/*eslint quotes: ["error", "single", { "avoidEscape": true }]*/

var double = "a string containing 'single' quotes";

Examples of additional correct code for this rule with the "backtick", { "avoidEscape": true } options:

/*eslint quotes: ["error", "backtick", { "avoidEscape": true }]*/

var double = "a string containing `backtick` quotes"

allowTemplateLiterals

Examples of additional correct code for this rule with the "double", { "allowTemplateLiterals": true } options:

/*eslint quotes: ["error", "double", { "allowTemplateLiterals": true }]*/

var double = "double";
var double = `double`;

Examples of additional correct code for this rule with the "single", { "allowTemplateLiterals": true } options:

/*eslint quotes: ["error", "single", { "allowTemplateLiterals": true }]*/

var single = 'single';
var single = `single`;

When Not To Use It

If you do not need consistency in your string styles, you can safely disable this rule. Source: http://eslint.org/docs/rules/

Adjoining classes: .panel-heading.actionitem-open
Open

.panel-heading.actionitem-open {
Severity: Minor
Found in client/stylesheets/4minitz.css by csslint

Adjoining classes: .panel-heading.bright
Open

.panel-heading.bright {
Severity: Minor
Found in client/stylesheets/4minitz.css by csslint

Element (label.checkbox-inline) is overqualified, just use .checkbox-inline without element name.
Open

.checkbox input[type=checkbox]:checked + .checkbox-material .check, label.checkbox-inline input[type=checkbox]:checked + .checkbox-material .check {
Severity: Minor
Found in client/stylesheets/4minitz.css by csslint

Duplicate property 'margin' found.
Open

    margin: 0;

Rule doesn't have all its properties in alphabetical order.
Open

    html {

Rule doesn't have all its properties in alphabetical order.
Open

    .hide-for-large {

Duplicate property 'Margin-left' found.
Open

        Margin-left: 33.33333% !important;

Duplicate property 'Margin-left' found.
Open

        Margin-left: 50% !important;

Adjoining classes: table.button.small-expand table
Open

    table.button.small-expand table,

TODO found
Open

        // TODO: Use ItemFactory to create info-/actionItem Object then we can use the isSticky-Method

TODO found
Open

     * TODO Reactive performance may be better if we only update one array element in DB
Severity: Minor
Found in imports/minutes.js by fixme

TODO found
Open

        // TODO: refactor topic class and make this method obsolete
Severity: Minor
Found in imports/meetingseries.js by fixme

TODO found
Open

                TODO: Find a better solution.

TODO found
Open

.tt-hint {  /* TODO */
Severity
Category
Status
Source
Language