Issues in icons.js (develop) - Issues in develop - aristath/kirki - Measure and Improve Code Quality continuously with Scrutinizer

Issues (377)

modules/icons/icons.js (9 issues)

Labels

Coding Style 9

Severity

Informational 9

Aristeides Stathopoulos 9

/* global kirkiIcons */

jQuery( document ).ready( function() {


	'use strict';

	if ( ! _.isUndefined( kirkiIcons.section ) ) {

		// Parse sections and add icons.
		_.each( kirkiIcons.section, function( icon, sectionID ) {
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
);

			// Add icons in list.
			jQuery( '#accordion-section-' + sectionID + ' > h3' ).addClass( 'dashicons-before ' + icon );

			// Add icons on titles when a section is open.
			jQuery( '#sub-accordion-section-' + sectionID + ' .customize-section-title > h3' ).append( '<span class="dashicons ' + icon + '" style="float:left;padding-right:.1em;padding-top:2px;"></span>' );
		} );
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.

	}

	if ( ! _.isUndefined( kirkiIcons.panel ) ) {

		_.each( kirkiIcons.panel, function( icon, panelID ) {
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
);

			// Add icons in lists & headers.
			jQuery( '#accordion-panel-' + panelID + ' > h3, #sub-accordion-panel-' + panelID + ' .panel-title' ).addClass( 'dashicons-before ' + icon );
		} );
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.

	}

} );
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.


1			/* global kirkiIcons */
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this You must use "/**" style comments for a file comment Loading history...
2			jQuery( document ).ready( function() {
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this The opening parenthesis of a multi-line function call should be the last content on the line. Loading history...
3
4			'use strict';
5
6			if ( ! _.isUndefined( kirkiIcons.section ) ) {
7
8			// Parse sections and add icons.
9			_.each( kirkiIcons.section, function( icon, sectionID ) {
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this The opening parenthesis of a multi-line function call should be the last content on the line. Loading history... Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this For multi-line function calls, each argument should be on a separate line. For a function calls that spawns multiple lines, the coding style suggests to split arguments to separate lines like this: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); Loading history...
10
11			// Add icons in list.
12			jQuery( '#accordion-section-' + sectionID + ' > h3' ).addClass( 'dashicons-before ' + icon );
13
14			// Add icons on titles when a section is open.
15			jQuery( '#sub-accordion-section-' + sectionID + ' .customize-section-title > h3' ).append( '<span class="dashicons ' + icon + '" style="float:left;padding-right:.1em;padding-top:2px;"></span>' );
16			} );
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
17
18			}
19
20			if ( ! _.isUndefined( kirkiIcons.panel ) ) {
21
22			_.each( kirkiIcons.panel, function( icon, panelID ) {
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this The opening parenthesis of a multi-line function call should be the last content on the line. Loading history... Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this For multi-line function calls, each argument should be on a separate line. For a function calls that spawns multiple lines, the coding style suggests to split arguments to separate lines like this: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); Loading history...
23
24			// Add icons in lists & headers.
25			jQuery( '#accordion-panel-' + panelID + ' > h3, #sub-accordion-panel-' + panelID + ' .panel-title' ).addClass( 'dashicons-before ' + icon );
26			} );
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
27
28			}
29
30			} );
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
31

aristath / kirki

Issues (377)

modules/icons/icons.js (9 issues)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like