tmcgee/cmv-widgets

View on GitHub

Showing 293 of 293 total issues

Function loadQueryBuilder has 402 lines of code (exceeds 100 allowed). Consider refactoring.
Open

        loadQueryBuilder: function () {
            var qbOptions = this.mixinDeep(this.defaultQueryBuilderOptions, this.queryBuilderOptions || {});

            // Namespace the configuration variables in this mixin
            this.queryBuilder = {
Severity: Major
Found in widgets/Search/QueryBuilder/QueryBuilderMixin.js - About 1 day to fix

Function setScaleRanges has 162 lines of code (exceeds 100 allowed). Consider refactoring.
Open

        setScaleRanges: function () {
            if (!this.mapSheetParams) {
                return;
            }

Severity: Major
Found in widgets/PrintPlus.js - About 3 hrs to fix

Function _onLayoutChange has 105 lines of code (exceeds 100 allowed). Consider refactoring.
Open

        _onLayoutChange: function (layout) {
            // Called with a boolean argument whenever the widget is opened or closed (true means it is open).
            // The layout dropdown calls the function with the name of the layout on the change event.

            // Set the cursor position to the left end of the value
Severity: Major
Found in widgets/PrintPlus.js - About 1 hr to fix

Values of 0 shouldn't have units specified.
Open

    background-position: 0px 8px;
Severity: Minor
Found in css/cmv.css by csslint

Duplicate property 'max-width' found.
Open

  max-width: initial;

Values of 0 shouldn't have units specified.
Open

  box-shadow: 0 2px 0px -0px #235677;

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

.introjs-bullets {

Standard property 'border-radius' should come after vendor-prefixed property '-moz-border-radius'.
Open

  -moz-border-radius: 10px;

Using width with padding can sometimes make elements larger than you expect.
Open

  padding: 10px;

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

.introjs-arrow.right {

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

html, body {
Severity: Minor
Found in css/cmv.css by csslint

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

.introjs-button {

Using height with padding-top can sometimes make elements larger than you expect.
Open

    padding-top: 4em;
Severity: Minor
Found in css/cmv.css by csslint

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

.introjs-button:hover {

Using width with border can sometimes make elements larger than you expect.
Open

    border: 1px solid #999;
Severity: Minor
Found in css/cmv.css by csslint

Use of !important
Open

    width: calc(100% - 20px) !important;
Severity: Minor
Found in widgets/HeatMap/css/Heatmap.css by csslint

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

.introjs-fixParent {

Using width with border can sometimes make elements larger than you expect.
Open

  border: 3px solid #DA4433;

Adjoining classes: .introjs-arrow.right
Open

.introjs-arrow.right {

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

.introjs-arrow.left {
Severity
Category
Status
Source
Language