2021-12-25 16:22:56 +02:00
|
|
|
import fs from 'node:fs';
|
|
|
|
import path from 'node:path';
|
|
|
|
import {
|
|
|
|
getDirnameFromImportMeta,
|
|
|
|
htmlFriendlyToTitle,
|
2022-09-24 17:29:43 +02:00
|
|
|
collator,
|
2023-04-19 15:23:13 +02:00
|
|
|
} from './sdk.mjs';
|
2021-12-25 16:22:56 +02:00
|
|
|
import svgpath from 'svgpath';
|
|
|
|
import svgPathBbox from 'svg-path-bbox';
|
|
|
|
import parsePath from 'svg-path-segments';
|
|
|
|
|
|
|
|
const __dirname = getDirnameFromImportMeta(import.meta.url);
|
|
|
|
const dataFile = path.join(__dirname, '_data', 'simple-icons.json');
|
|
|
|
const htmlNamedEntitiesFile = path.join(
|
|
|
|
__dirname,
|
|
|
|
'node_modules',
|
|
|
|
'named-html-entities-json',
|
|
|
|
'index.json',
|
|
|
|
);
|
|
|
|
const svglintIgnoredFile = path.join(__dirname, '.svglint-ignored.json');
|
|
|
|
|
|
|
|
const data = JSON.parse(fs.readFileSync(dataFile, 'utf8'));
|
|
|
|
const htmlNamedEntities = JSON.parse(
|
|
|
|
fs.readFileSync(htmlNamedEntitiesFile, 'utf8'),
|
|
|
|
);
|
|
|
|
const svglintIgnores = JSON.parse(fs.readFileSync(svglintIgnoredFile, 'utf8'));
|
2021-10-25 21:13:10 +02:00
|
|
|
|
|
|
|
const svgRegexp =
|
2022-07-04 18:47:33 +02:00
|
|
|
/^<svg( [^\s]*=".*"){3}><title>.*<\/title><path d=".*"\/><\/svg>$/;
|
2020-12-14 21:35:27 +02:00
|
|
|
const negativeZerosRegexp = /-0(?=[^\.]|[\s\d\w]|$)/g;
|
2020-06-22 18:24:56 +02:00
|
|
|
|
2020-06-10 11:59:42 +02:00
|
|
|
const iconSize = 24;
|
|
|
|
const iconFloatPrecision = 3;
|
2020-11-19 22:49:49 +02:00
|
|
|
const iconMaxFloatPrecision = 5;
|
2020-07-10 10:06:08 +02:00
|
|
|
const iconTolerance = 0.001;
|
2020-10-17 13:01:56 +02:00
|
|
|
|
|
|
|
// set env SI_UPDATE_IGNORE to recreate the ignore file
|
2021-01-15 22:47:00 +02:00
|
|
|
const updateIgnoreFile = process.env.SI_UPDATE_IGNORE === 'true';
|
2021-10-25 21:13:10 +02:00
|
|
|
const ignoreFile = './.svglint-ignored.json';
|
2021-12-25 16:22:56 +02:00
|
|
|
const iconIgnored = !updateIgnoreFile ? svglintIgnores : {};
|
2020-10-17 13:01:56 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const sortObjectByKey = (obj) => {
|
|
|
|
return Object.keys(obj)
|
2020-10-17 13:01:56 +02:00
|
|
|
.sort()
|
|
|
|
.reduce((r, k) => Object.assign(r, { [k]: obj[k] }), {});
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2020-10-17 13:01:56 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const sortObjectByValue = (obj) => {
|
|
|
|
return Object.keys(obj)
|
2022-09-24 17:29:43 +02:00
|
|
|
.sort((a, b) => collator.compare(obj[a], obj[b]))
|
2020-10-17 13:01:56 +02:00
|
|
|
.reduce((r, k) => Object.assign(r, { [k]: obj[k] }), {});
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2020-10-17 13:01:56 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const removeLeadingZeros = (number) => {
|
2020-12-13 21:29:01 +02:00
|
|
|
// convert 0.03 to '.03'
|
|
|
|
return number.toString().replace(/^(-?)(0)(\.?.+)/, '$1$3');
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2020-12-13 21:29:01 +02:00
|
|
|
|
2021-01-03 19:08:06 +02:00
|
|
|
/**
|
|
|
|
* Given three points, returns if the middle one (x2, y2) is collinear
|
|
|
|
* to the line formed by the two limit points.
|
|
|
|
**/
|
2021-10-25 21:13:10 +02:00
|
|
|
const collinear = (x1, y1, x2, y2, x3, y3) => {
|
|
|
|
return x1 * (y2 - y3) + x2 * (y3 - y1) + x3 * (y1 - y2) === 0;
|
|
|
|
};
|
2021-01-03 19:08:06 +02:00
|
|
|
|
2021-03-02 20:00:18 +02:00
|
|
|
/**
|
|
|
|
* Returns the number of digits after the decimal point.
|
|
|
|
* @param num The number of interest.
|
|
|
|
*/
|
2021-10-25 21:13:10 +02:00
|
|
|
const countDecimals = (num) => {
|
2021-03-02 20:00:18 +02:00
|
|
|
if (num && num % 1) {
|
|
|
|
let [base, op, trail] = num.toExponential().split(/e([+-])/);
|
|
|
|
let elen = parseInt(trail, 10);
|
|
|
|
let idx = base.indexOf('.');
|
2021-10-25 21:13:10 +02:00
|
|
|
return idx == -1
|
|
|
|
? elen
|
|
|
|
: base.length - idx - 1 + (op === '+' ? -elen : elen);
|
2021-03-02 20:00:18 +02:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Get the index at which the first path value of an SVG starts.
|
|
|
|
* @param svgFileContent The raw SVG as text.
|
|
|
|
*/
|
2021-10-25 21:13:10 +02:00
|
|
|
const getPathDIndex = (svgFileContent) => {
|
2021-03-02 20:00:18 +02:00
|
|
|
const pathDStart = '<path d="';
|
|
|
|
return svgFileContent.indexOf(pathDStart) + pathDStart.length;
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2021-03-02 20:00:18 +02:00
|
|
|
|
2021-10-02 16:46:07 +02:00
|
|
|
/**
|
|
|
|
* Get the index at which the text of the first `<title></title>` tag starts.
|
|
|
|
* @param svgFileContent The raw SVG as text.
|
|
|
|
**/
|
2021-10-25 21:13:10 +02:00
|
|
|
const getTitleTextIndex = (svgFileContent) => {
|
2021-10-02 16:46:07 +02:00
|
|
|
const titleStart = '<title>';
|
|
|
|
return svgFileContent.indexOf(titleStart) + titleStart.length;
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2021-10-02 16:46:07 +02:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Convert a hexadecimal number passed as string to decimal number as integer.
|
|
|
|
* @param hex The hexadecimal number representation to convert.
|
|
|
|
**/
|
2021-10-25 21:13:10 +02:00
|
|
|
const hexadecimalToDecimal = (hex) => {
|
|
|
|
let result = 0,
|
|
|
|
digitValue;
|
2021-10-02 16:46:07 +02:00
|
|
|
hex = hex.toLowerCase();
|
|
|
|
for (var i = 0; i < hex.length; i++) {
|
|
|
|
digitValue = '0123456789abcdefgh'.indexOf(hex[i]);
|
|
|
|
result = result * 16 + digitValue;
|
|
|
|
}
|
|
|
|
return result;
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2020-10-17 13:01:56 +02:00
|
|
|
if (updateIgnoreFile) {
|
|
|
|
process.on('exit', () => {
|
|
|
|
// ensure object output order is consistent due to async svglint processing
|
2021-10-25 21:13:10 +02:00
|
|
|
const sorted = sortObjectByKey(iconIgnored);
|
2020-10-17 13:01:56 +02:00
|
|
|
for (const linterName in sorted) {
|
2021-10-25 21:13:10 +02:00
|
|
|
sorted[linterName] = sortObjectByValue(sorted[linterName]);
|
2020-10-17 13:01:56 +02:00
|
|
|
}
|
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
fs.writeFileSync(ignoreFile, JSON.stringify(sorted, null, 2) + '\n', {
|
|
|
|
flag: 'w',
|
|
|
|
});
|
2020-10-17 13:01:56 +02:00
|
|
|
});
|
|
|
|
}
|
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const isIgnored = (linterName, path) => {
|
|
|
|
return (
|
|
|
|
iconIgnored[linterName] && iconIgnored[linterName].hasOwnProperty(path)
|
|
|
|
);
|
|
|
|
};
|
2020-10-17 13:01:56 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const ignoreIcon = (linterName, path, $) => {
|
2020-10-17 13:01:56 +02:00
|
|
|
if (!iconIgnored[linterName]) {
|
|
|
|
iconIgnored[linterName] = {};
|
|
|
|
}
|
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const title = $.find('title').text();
|
2020-10-17 13:01:56 +02:00
|
|
|
const iconName = htmlFriendlyToTitle(title);
|
|
|
|
|
|
|
|
iconIgnored[linterName][path] = iconName;
|
2021-10-25 21:13:10 +02:00
|
|
|
};
|
2019-07-03 23:33:03 +02:00
|
|
|
|
2021-12-25 16:22:56 +02:00
|
|
|
export default {
|
2021-10-25 21:13:10 +02:00
|
|
|
rules: {
|
|
|
|
elm: {
|
|
|
|
svg: 1,
|
|
|
|
'svg > title': 1,
|
|
|
|
'svg > path': 1,
|
|
|
|
'*': false,
|
|
|
|
},
|
|
|
|
attr: [
|
|
|
|
{
|
2021-12-02 01:03:52 +02:00
|
|
|
// ensure that the SVG elm has the appropriate attrs alphabetically ordered
|
2021-10-25 21:13:10 +02:00
|
|
|
role: 'img',
|
|
|
|
viewBox: `0 0 ${iconSize} ${iconSize}`,
|
|
|
|
xmlns: 'http://www.w3.org/2000/svg',
|
|
|
|
'rule::selector': 'svg',
|
|
|
|
'rule::whitelist': true,
|
2021-12-02 01:03:52 +02:00
|
|
|
'rule::order': true,
|
2021-10-25 21:13:10 +02:00
|
|
|
},
|
|
|
|
{
|
|
|
|
// ensure that the title elm has the appropriate attr
|
|
|
|
'rule::selector': 'svg > title',
|
|
|
|
'rule::whitelist': true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// ensure that the path element only has the 'd' attr (no style, opacity, etc.)
|
|
|
|
d: /^[,a-zA-Z0-9\. -]+$/,
|
|
|
|
'rule::selector': 'svg > path',
|
|
|
|
'rule::whitelist': true,
|
|
|
|
},
|
|
|
|
],
|
|
|
|
custom: [
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'icon-title';
|
|
|
|
|
|
|
|
const iconTitleText = $.find('title').text(),
|
|
|
|
xmlNamedEntitiesCodepoints = [38, 60, 62],
|
|
|
|
xmlNamedEntities = ['amp', 'lt', 'gt'];
|
|
|
|
let _validCodepointsRepr = true;
|
|
|
|
|
|
|
|
// avoid character codepoints as hexadecimal representation
|
|
|
|
const hexadecimalCodepoints = Array.from(
|
|
|
|
iconTitleText.matchAll(/&#x([A-Fa-f0-9]+);/g),
|
|
|
|
);
|
|
|
|
if (hexadecimalCodepoints.length > 0) {
|
|
|
|
_validCodepointsRepr = false;
|
|
|
|
|
|
|
|
hexadecimalCodepoints.forEach((match) => {
|
|
|
|
const charHexReprIndex =
|
|
|
|
getTitleTextIndex(ast.source) + match.index + 1;
|
|
|
|
const charDec = hexadecimalToDecimal(match[1]);
|
|
|
|
|
|
|
|
let charRepr;
|
|
|
|
if (xmlNamedEntitiesCodepoints.includes(charDec)) {
|
|
|
|
charRepr = `&${
|
|
|
|
xmlNamedEntities[xmlNamedEntitiesCodepoints.indexOf(charDec)]
|
|
|
|
};`;
|
|
|
|
} else if (charDec < 128) {
|
|
|
|
charRepr = String.fromCodePoint(charDec);
|
|
|
|
} else {
|
|
|
|
charRepr = `&#${charDec};`;
|
2018-08-16 11:33:32 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
|
|
|
|
reporter.error(
|
|
|
|
`Hexadecimal representation of encoded character "${match[0]}" found at index ${charHexReprIndex}:` +
|
|
|
|
` replace it with "${charRepr}".`,
|
2021-10-02 16:46:07 +02:00
|
|
|
);
|
2021-10-25 21:13:10 +02:00
|
|
|
});
|
|
|
|
}
|
|
|
|
|
|
|
|
// avoid character codepoints as named entities
|
|
|
|
const namedEntitiesCodepoints = Array.from(
|
|
|
|
iconTitleText.matchAll(/&([A-Za-z0-9]+);/g),
|
|
|
|
);
|
|
|
|
if (namedEntitiesCodepoints.length > 0) {
|
|
|
|
namedEntitiesCodepoints.forEach((match) => {
|
|
|
|
const namedEntiyReprIndex =
|
|
|
|
getTitleTextIndex(ast.source) + match.index + 1;
|
|
|
|
|
|
|
|
if (!xmlNamedEntities.includes(match[1].toLowerCase())) {
|
2021-10-02 16:46:07 +02:00
|
|
|
_validCodepointsRepr = false;
|
2021-10-25 21:13:10 +02:00
|
|
|
const namedEntityJsRepr = htmlNamedEntities[match[1]];
|
|
|
|
let replacement;
|
|
|
|
|
|
|
|
if (
|
|
|
|
namedEntityJsRepr === undefined ||
|
|
|
|
namedEntityJsRepr.length != 1
|
|
|
|
) {
|
|
|
|
replacement = 'its decimal or literal representation';
|
|
|
|
} else {
|
|
|
|
const namedEntityDec = namedEntityJsRepr.codePointAt(0);
|
|
|
|
if (namedEntityDec < 128) {
|
|
|
|
replacement = `"${namedEntityJsRepr}"`;
|
2021-10-02 16:46:07 +02:00
|
|
|
} else {
|
2021-10-25 21:13:10 +02:00
|
|
|
replacement = `"&#${namedEntityDec};"`;
|
2021-10-02 16:46:07 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
reporter.error(
|
|
|
|
`Named entity representation of encoded character "${match[0]}" found at index ${namedEntiyReprIndex}.` +
|
|
|
|
` Replace it with ${replacement}.`,
|
|
|
|
);
|
2021-10-02 16:46:07 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
});
|
|
|
|
}
|
|
|
|
|
|
|
|
if (_validCodepointsRepr) {
|
|
|
|
// compare encoded title with original title and report error if not equal
|
|
|
|
const encodingMatches = Array.from(
|
|
|
|
iconTitleText.matchAll(/&(#([0-9]+)|(amp|quot|lt|gt));/g),
|
|
|
|
),
|
|
|
|
encodedBuf = [];
|
|
|
|
|
|
|
|
const _indexesToIgnore = [];
|
|
|
|
for (let m = 0; m < encodingMatches.length; m++) {
|
|
|
|
let index = encodingMatches[m].index;
|
|
|
|
for (let r = index; r < index + encodingMatches[m][0].length; r++) {
|
|
|
|
_indexesToIgnore.push(r);
|
|
|
|
}
|
|
|
|
}
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
for (let i = iconTitleText.length - 1; i >= 0; i--) {
|
|
|
|
if (_indexesToIgnore.includes(i)) {
|
|
|
|
encodedBuf.unshift(iconTitleText[i]);
|
|
|
|
} else {
|
|
|
|
// encode all non ascii characters plus "'&<> (XML named entities)
|
|
|
|
let charDecimalCode = iconTitleText.charCodeAt(i);
|
|
|
|
|
|
|
|
if (charDecimalCode > 127) {
|
|
|
|
encodedBuf.unshift(`&#${charDecimalCode};`);
|
|
|
|
} else if (xmlNamedEntitiesCodepoints.includes(charDecimalCode)) {
|
|
|
|
encodedBuf.unshift(
|
|
|
|
`&${
|
|
|
|
xmlNamedEntities[
|
|
|
|
xmlNamedEntitiesCodepoints.indexOf(charDecimalCode)
|
|
|
|
]
|
|
|
|
};`,
|
|
|
|
);
|
|
|
|
} else {
|
|
|
|
encodedBuf.unshift(iconTitleText[i]);
|
|
|
|
}
|
2021-10-02 16:46:07 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
const encodedIconTitleText = encodedBuf.join('');
|
|
|
|
if (encodedIconTitleText !== iconTitleText) {
|
|
|
|
_validCodepointsRepr = false;
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
reporter.error(
|
|
|
|
`Unencoded unicode characters found in title "${iconTitleText}":` +
|
|
|
|
` rewrite it as "${encodedIconTitleText}".`,
|
|
|
|
);
|
|
|
|
}
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
// check if there are some other encoded characters in decimal notation
|
|
|
|
// which shouldn't be encoded
|
|
|
|
encodingMatches
|
|
|
|
.filter((m) => !isNaN(m[2]))
|
|
|
|
.forEach((match) => {
|
|
|
|
const decimalNumber = parseInt(match[2]);
|
|
|
|
if (decimalNumber < 128) {
|
|
|
|
_validCodepointsRepr = false;
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
const decimalCodepointCharIndex =
|
|
|
|
getTitleTextIndex(ast.source) + match.index + 1;
|
|
|
|
if (xmlNamedEntitiesCodepoints.includes(decimalNumber)) {
|
|
|
|
replacement = `"&${
|
|
|
|
xmlNamedEntities[
|
|
|
|
xmlNamedEntitiesCodepoints.indexOf(decimalNumber)
|
|
|
|
]
|
|
|
|
};"`;
|
2021-10-02 16:46:07 +02:00
|
|
|
} else {
|
2021-10-25 21:13:10 +02:00
|
|
|
replacement = String.fromCharCode(decimalNumber);
|
|
|
|
replacement = replacement == '"' ? `'"'` : `"${replacement}"`;
|
2021-10-02 16:46:07 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
reporter.error(
|
2021-10-25 21:13:10 +02:00
|
|
|
`Unnecessary encoded character "${match[0]}" found at index ${decimalCodepointCharIndex}:` +
|
|
|
|
` replace it with ${replacement}.`,
|
2021-10-02 16:46:07 +02:00
|
|
|
);
|
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
});
|
2021-10-02 16:46:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (_validCodepointsRepr) {
|
|
|
|
const iconName = htmlFriendlyToTitle(iconTitleText);
|
|
|
|
const iconExists = data.icons.some(
|
|
|
|
(icon) => icon.title === iconName,
|
|
|
|
);
|
|
|
|
if (!iconExists) {
|
|
|
|
reporter.error(
|
|
|
|
`No icon with title "${iconName}" found in simple-icons.json`,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'icon-size';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
const [minX, minY, maxX, maxY] = svgPathBbox(iconPath);
|
|
|
|
const width = +(maxX - minX).toFixed(iconFloatPrecision);
|
|
|
|
const height = +(maxY - minY).toFixed(iconFloatPrecision);
|
|
|
|
|
|
|
|
if (width === 0 && height === 0) {
|
|
|
|
reporter.error(
|
|
|
|
'Path bounds were reported as 0 x 0; check if the path is valid',
|
|
|
|
);
|
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
|
|
|
}
|
|
|
|
} else if (width !== iconSize && height !== iconSize) {
|
|
|
|
reporter.error(
|
|
|
|
`Size of <path> must be exactly ${iconSize} in one dimension; the size is currently ${width} x ${height}`,
|
|
|
|
);
|
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'icon-precision';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
const segments = parsePath(iconPath),
|
|
|
|
svgFileContent = $.html();
|
|
|
|
|
|
|
|
segments.forEach((segment) => {
|
|
|
|
const precisionMax = Math.max(
|
|
|
|
...segment.params.slice(1).map(countDecimals),
|
|
|
|
);
|
|
|
|
if (precisionMax > iconMaxFloatPrecision) {
|
|
|
|
let errorMsg = `found ${precisionMax} decimals in segment "${iconPath.substring(
|
|
|
|
segment.start,
|
|
|
|
segment.end,
|
|
|
|
)}"`;
|
|
|
|
if (segment.chained) {
|
|
|
|
let readableChain = iconPath.substring(
|
|
|
|
segment.chainStart,
|
|
|
|
segment.chainEnd,
|
|
|
|
);
|
|
|
|
if (readableChain.length > 20) {
|
|
|
|
readableChain = `${readableChain.substring(0, 20)}...`;
|
2021-10-02 16:46:07 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
errorMsg += ` of chain "${readableChain}"`;
|
2019-07-03 23:33:03 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
errorMsg += ` at index ${
|
|
|
|
segment.start + getPathDIndex(svgFileContent)
|
|
|
|
}`;
|
|
|
|
reporter.error(
|
|
|
|
`Maximum precision should not be greater than ${iconMaxFloatPrecision}; ${errorMsg}`,
|
|
|
|
);
|
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
2020-06-10 11:59:42 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
});
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'ineffective-segments';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
const segments = parsePath(iconPath);
|
|
|
|
const absSegments = svgpath(iconPath).abs().unshort().segments;
|
|
|
|
|
|
|
|
const lowerMovementCommands = ['m', 'l'];
|
|
|
|
const lowerDirectionCommands = ['h', 'v'];
|
|
|
|
const lowerCurveCommand = 'c';
|
|
|
|
const lowerShorthandCurveCommand = 's';
|
|
|
|
const lowerCurveCommands = [
|
|
|
|
lowerCurveCommand,
|
|
|
|
lowerShorthandCurveCommand,
|
|
|
|
];
|
|
|
|
const upperMovementCommands = ['M', 'L'];
|
|
|
|
const upperHorDirectionCommand = 'H';
|
|
|
|
const upperVerDirectionCommand = 'V';
|
|
|
|
const upperDirectionCommands = [
|
|
|
|
upperHorDirectionCommand,
|
|
|
|
upperVerDirectionCommand,
|
|
|
|
];
|
|
|
|
const upperCurveCommand = 'C';
|
|
|
|
const upperShorthandCurveCommand = 'S';
|
|
|
|
const upperCurveCommands = [
|
|
|
|
upperCurveCommand,
|
|
|
|
upperShorthandCurveCommand,
|
|
|
|
];
|
|
|
|
const curveCommands = [...lowerCurveCommands, ...upperCurveCommands];
|
|
|
|
const commands = [
|
|
|
|
...lowerMovementCommands,
|
|
|
|
...lowerDirectionCommands,
|
|
|
|
...upperMovementCommands,
|
|
|
|
...upperDirectionCommands,
|
|
|
|
...curveCommands,
|
|
|
|
];
|
|
|
|
const isInvalidSegment = (
|
|
|
|
[command, x1Coord, y1Coord, ...rest],
|
|
|
|
index,
|
|
|
|
) => {
|
|
|
|
if (commands.includes(command)) {
|
|
|
|
// Relative directions (h or v) having a length of 0
|
|
|
|
if (lowerDirectionCommands.includes(command) && x1Coord === 0) {
|
|
|
|
return true;
|
2020-06-10 11:59:42 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
// Relative movement (m or l) having a distance of 0
|
|
|
|
if (
|
|
|
|
index > 0 &&
|
|
|
|
lowerMovementCommands.includes(command) &&
|
|
|
|
x1Coord === 0 &&
|
|
|
|
y1Coord === 0
|
|
|
|
) {
|
|
|
|
return true;
|
2020-11-19 22:49:49 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
if (
|
|
|
|
lowerCurveCommands.includes(command) &&
|
|
|
|
x1Coord === 0 &&
|
|
|
|
y1Coord === 0
|
|
|
|
) {
|
|
|
|
const [x2Coord, y2Coord] = rest;
|
|
|
|
if (
|
|
|
|
// Relative shorthand curve (s) having a control point of 0
|
|
|
|
command === lowerShorthandCurveCommand ||
|
|
|
|
// Relative bézier curve (c) having control points of 0
|
|
|
|
(command === lowerCurveCommand &&
|
|
|
|
x2Coord === 0 &&
|
|
|
|
y2Coord === 0)
|
|
|
|
) {
|
|
|
|
return true;
|
2020-11-19 22:49:49 +02:00
|
|
|
}
|
2020-12-13 21:29:01 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
if (index > 0) {
|
|
|
|
let [yPrevCoord, xPrevCoord] = [
|
|
|
|
...absSegments[index - 1],
|
|
|
|
].reverse();
|
|
|
|
// If the previous command was a direction one, we need to iterate back until we find the missing coordinates
|
|
|
|
if (upperDirectionCommands.includes(xPrevCoord)) {
|
|
|
|
xPrevCoord = undefined;
|
|
|
|
yPrevCoord = undefined;
|
|
|
|
let idx = index;
|
|
|
|
while (
|
|
|
|
--idx > 0 &&
|
|
|
|
(xPrevCoord === undefined || yPrevCoord === undefined)
|
|
|
|
) {
|
|
|
|
let [yPrevCoordDeep, xPrevCoordDeep] = [
|
|
|
|
...absSegments[idx],
|
|
|
|
].reverse();
|
|
|
|
// If the previous command was a horizontal movement, we need to consider the single coordinate as x
|
|
|
|
if (upperHorDirectionCommand === xPrevCoordDeep) {
|
|
|
|
xPrevCoordDeep = yPrevCoordDeep;
|
|
|
|
yPrevCoordDeep = undefined;
|
2020-12-13 21:29:01 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
// If the previous command was a vertical movement, we need to consider the single coordinate as y
|
|
|
|
if (upperVerDirectionCommand === xPrevCoordDeep) {
|
|
|
|
xPrevCoordDeep = undefined;
|
2020-12-13 21:29:01 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
if (
|
|
|
|
xPrevCoord === undefined &&
|
|
|
|
xPrevCoordDeep !== undefined
|
|
|
|
) {
|
|
|
|
xPrevCoord = xPrevCoordDeep;
|
2020-12-13 21:29:01 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
if (
|
|
|
|
yPrevCoord === undefined &&
|
|
|
|
yPrevCoordDeep !== undefined
|
|
|
|
) {
|
|
|
|
yPrevCoord = yPrevCoordDeep;
|
2021-03-02 20:00:18 +02:00
|
|
|
}
|
2020-12-13 21:29:01 +02:00
|
|
|
}
|
|
|
|
}
|
2021-01-15 22:47:00 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (upperCurveCommands.includes(command)) {
|
|
|
|
const [x2Coord, y2Coord, xCoord, yCoord] = rest;
|
|
|
|
// Absolute shorthand curve (S) having the same coordinate as the previous segment and a control point equal to the ending point
|
|
|
|
if (
|
|
|
|
upperShorthandCurveCommand === command &&
|
|
|
|
x1Coord === xPrevCoord &&
|
|
|
|
y1Coord === yPrevCoord &&
|
|
|
|
x1Coord === x2Coord &&
|
|
|
|
y1Coord === y2Coord
|
|
|
|
) {
|
|
|
|
return true;
|
2021-01-15 12:16:50 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
// Absolute bézier curve (C) having the same coordinate as the previous segment and last control point equal to the ending point
|
|
|
|
if (
|
|
|
|
upperCurveCommand === command &&
|
|
|
|
x1Coord === xPrevCoord &&
|
|
|
|
y1Coord === yPrevCoord &&
|
|
|
|
x2Coord === xCoord &&
|
|
|
|
y2Coord === yCoord
|
|
|
|
) {
|
|
|
|
return true;
|
2021-01-03 19:08:06 +02:00
|
|
|
}
|
|
|
|
}
|
2021-01-15 22:47:00 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
return (
|
|
|
|
// Absolute horizontal direction (H) having the same x coordinate as the previous segment
|
|
|
|
(upperHorDirectionCommand === command &&
|
|
|
|
x1Coord === xPrevCoord) ||
|
|
|
|
// Absolute vertical direction (V) having the same y coordinate as the previous segment
|
|
|
|
(upperVerDirectionCommand === command &&
|
|
|
|
x1Coord === yPrevCoord) ||
|
|
|
|
// Absolute movement (M or L) having the same coordinate as the previous segment
|
|
|
|
(upperMovementCommands.includes(command) &&
|
|
|
|
x1Coord === xPrevCoord &&
|
|
|
|
y1Coord === yPrevCoord)
|
|
|
|
);
|
2021-01-03 19:08:06 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
const svgFileContent = $.html();
|
|
|
|
|
|
|
|
segments.forEach((segment, index) => {
|
|
|
|
if (isInvalidSegment(segment.params, index)) {
|
|
|
|
const [command, x1, y1, ...rest] = segment.params;
|
|
|
|
|
|
|
|
let errorMsg = `Innefective segment "${iconPath.substring(
|
|
|
|
segment.start,
|
|
|
|
segment.end,
|
|
|
|
)}" found`,
|
|
|
|
resolutionTip = 'should be removed';
|
|
|
|
|
|
|
|
if (curveCommands.includes(command)) {
|
|
|
|
const [x2, y2, x, y] = rest;
|
|
|
|
|
|
|
|
if (
|
|
|
|
command === lowerShorthandCurveCommand &&
|
|
|
|
(x2 !== 0 || y2 !== 0)
|
|
|
|
) {
|
|
|
|
resolutionTip = `should be "l${removeLeadingZeros(
|
|
|
|
x2,
|
|
|
|
)} ${removeLeadingZeros(y2)}" or removed`;
|
2021-01-03 19:08:06 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
if (command === upperShorthandCurveCommand) {
|
|
|
|
resolutionTip = `should be "L${removeLeadingZeros(
|
|
|
|
x2,
|
|
|
|
)} ${removeLeadingZeros(y2)}" or removed`;
|
|
|
|
}
|
|
|
|
if (command === lowerCurveCommand && (x !== 0 || y !== 0)) {
|
|
|
|
resolutionTip = `should be "l${removeLeadingZeros(
|
|
|
|
x,
|
|
|
|
)} ${removeLeadingZeros(y)}" or removed`;
|
|
|
|
}
|
|
|
|
if (command === upperCurveCommand) {
|
|
|
|
resolutionTip = `should be "L${removeLeadingZeros(
|
|
|
|
x,
|
|
|
|
)} ${removeLeadingZeros(y)}" or removed`;
|
2021-03-02 20:00:18 +02:00
|
|
|
}
|
|
|
|
}
|
2020-06-22 18:24:56 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (segment.chained) {
|
|
|
|
let readableChain = iconPath.substring(
|
|
|
|
segment.chainStart,
|
|
|
|
segment.chainEnd,
|
|
|
|
);
|
|
|
|
if (readableChain.length > 20) {
|
2021-11-10 02:12:10 +02:00
|
|
|
readableChain = `${readableChain.substring(0, 20)}...`;
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
errorMsg += ` in chain "${readableChain}"`;
|
2020-06-22 18:24:56 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
errorMsg += ` at index ${
|
|
|
|
segment.start + getPathDIndex(svgFileContent)
|
|
|
|
}`;
|
2020-12-14 21:35:27 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
reporter.error(`${errorMsg} (${resolutionTip})`);
|
2020-12-14 21:35:27 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
2020-12-14 21:35:27 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
});
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'collinear-segments';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Extracts collinear coordinates from SVG path straight lines
|
|
|
|
* (does not extracts collinear coordinates from curves).
|
|
|
|
**/
|
|
|
|
const getCollinearSegments = (iconPath) => {
|
|
|
|
const segments = parsePath(iconPath),
|
|
|
|
collinearSegments = [],
|
|
|
|
straightLineCommands = 'HhVvLlMm',
|
|
|
|
zCommands = 'Zz';
|
|
|
|
|
|
|
|
let currLine = [],
|
|
|
|
currAbsCoord = [undefined, undefined],
|
|
|
|
startPoint,
|
|
|
|
_inStraightLine = false,
|
|
|
|
_nextInStraightLine = false,
|
|
|
|
_resetStartPoint = false;
|
|
|
|
|
|
|
|
for (let s = 0; s < segments.length; s++) {
|
|
|
|
let seg = segments[s].params,
|
|
|
|
cmd = seg[0],
|
|
|
|
nextCmd = s + 1 < segments.length ? segments[s + 1][0] : null;
|
|
|
|
|
|
|
|
if (cmd === 'L') {
|
|
|
|
currAbsCoord[0] = seg[1];
|
|
|
|
currAbsCoord[1] = seg[2];
|
|
|
|
} else if (cmd === 'l') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[1];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[2];
|
|
|
|
} else if (cmd === 'm') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[1];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[2];
|
|
|
|
startPoint = undefined;
|
|
|
|
} else if (cmd === 'M') {
|
|
|
|
currAbsCoord[0] = seg[1];
|
|
|
|
currAbsCoord[1] = seg[2];
|
|
|
|
startPoint = undefined;
|
|
|
|
} else if (cmd === 'H') {
|
|
|
|
currAbsCoord[0] = seg[1];
|
|
|
|
} else if (cmd === 'h') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[1];
|
|
|
|
} else if (cmd === 'V') {
|
|
|
|
currAbsCoord[1] = seg[1];
|
|
|
|
} else if (cmd === 'v') {
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[1];
|
|
|
|
} else if (cmd === 'C') {
|
|
|
|
currAbsCoord[0] = seg[5];
|
|
|
|
currAbsCoord[1] = seg[6];
|
|
|
|
} else if (cmd === 'a') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[6];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[7];
|
|
|
|
} else if (cmd === 'A') {
|
|
|
|
currAbsCoord[0] = seg[6];
|
|
|
|
currAbsCoord[1] = seg[7];
|
|
|
|
} else if (cmd === 's') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[1];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[2];
|
|
|
|
} else if (cmd === 'S') {
|
|
|
|
currAbsCoord[0] = seg[1];
|
|
|
|
currAbsCoord[1] = seg[2];
|
|
|
|
} else if (cmd === 't') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[1];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[2];
|
|
|
|
} else if (cmd === 'T') {
|
|
|
|
currAbsCoord[0] = seg[1];
|
|
|
|
currAbsCoord[1] = seg[2];
|
|
|
|
} else if (cmd === 'c') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[5];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[6];
|
|
|
|
} else if (cmd === 'Q') {
|
|
|
|
currAbsCoord[0] = seg[3];
|
|
|
|
currAbsCoord[1] = seg[4];
|
|
|
|
} else if (cmd === 'q') {
|
|
|
|
currAbsCoord[0] =
|
|
|
|
(!currAbsCoord[0] ? 0 : currAbsCoord[0]) + seg[3];
|
|
|
|
currAbsCoord[1] =
|
|
|
|
(!currAbsCoord[1] ? 0 : currAbsCoord[1]) + seg[4];
|
|
|
|
} else if (zCommands.includes(cmd)) {
|
|
|
|
// Overlapping in Z should be handled in another rule
|
|
|
|
currAbsCoord = [startPoint[0], startPoint[1]];
|
|
|
|
_resetStartPoint = true;
|
|
|
|
} else {
|
|
|
|
throw new Error(`"${cmd}" command not handled`);
|
2020-07-10 10:06:08 +02:00
|
|
|
}
|
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (startPoint === undefined) {
|
|
|
|
startPoint = [currAbsCoord[0], currAbsCoord[1]];
|
|
|
|
} else if (_resetStartPoint) {
|
|
|
|
startPoint = undefined;
|
|
|
|
_resetStartPoint = false;
|
2020-07-10 10:06:08 +02:00
|
|
|
}
|
2021-02-22 18:20:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
_nextInStraightLine = straightLineCommands.includes(nextCmd);
|
|
|
|
let _exitingStraightLine = _inStraightLine && !_nextInStraightLine;
|
|
|
|
_inStraightLine = straightLineCommands.includes(cmd);
|
2021-02-22 18:20:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
if (_inStraightLine) {
|
|
|
|
currLine.push([currAbsCoord[0], currAbsCoord[1]]);
|
|
|
|
} else {
|
|
|
|
if (_exitingStraightLine) {
|
|
|
|
if (straightLineCommands.includes(cmd)) {
|
|
|
|
currLine.push([currAbsCoord[0], currAbsCoord[1]]);
|
|
|
|
}
|
|
|
|
// Get collinear coordinates
|
|
|
|
for (let p = 1; p < currLine.length - 1; p++) {
|
|
|
|
let _collinearCoord = collinear(
|
|
|
|
currLine[p - 1][0],
|
|
|
|
currLine[p - 1][1],
|
|
|
|
currLine[p][0],
|
|
|
|
currLine[p][1],
|
|
|
|
currLine[p + 1][0],
|
|
|
|
currLine[p + 1][1],
|
|
|
|
);
|
|
|
|
if (_collinearCoord) {
|
|
|
|
collinearSegments.push(
|
|
|
|
segments[s - currLine.length + p + 1],
|
|
|
|
);
|
|
|
|
}
|
2021-02-22 18:20:07 +02:00
|
|
|
}
|
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
currLine = [];
|
|
|
|
}
|
|
|
|
}
|
2021-02-22 18:20:07 +02:00
|
|
|
|
2021-10-25 21:13:10 +02:00
|
|
|
return collinearSegments;
|
|
|
|
};
|
|
|
|
|
|
|
|
const collinearSegments = getCollinearSegments(iconPath),
|
|
|
|
pathDIndex = getPathDIndex($.html());
|
|
|
|
collinearSegments.forEach((segment) => {
|
|
|
|
let errorMsg = `Collinear segment "${iconPath.substring(
|
|
|
|
segment.start,
|
|
|
|
segment.end,
|
|
|
|
)}" found`;
|
|
|
|
if (segment.chained) {
|
|
|
|
let readableChain = iconPath.substring(
|
|
|
|
segment.chainStart,
|
|
|
|
segment.chainEnd,
|
|
|
|
);
|
|
|
|
if (readableChain.length > 20) {
|
|
|
|
readableChain = `${readableChain.substring(0, 20)}...`;
|
2021-02-22 18:20:07 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
errorMsg += ` in chain "${readableChain}"`;
|
|
|
|
}
|
|
|
|
errorMsg += ` at index ${
|
|
|
|
segment.start + pathDIndex
|
|
|
|
} (should be removed)`;
|
|
|
|
reporter.error(errorMsg);
|
|
|
|
});
|
|
|
|
|
|
|
|
if (collinearSegments.length) {
|
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'extraneous';
|
|
|
|
|
|
|
|
if (!svgRegexp.test(ast.source)) {
|
2023-02-09 21:47:14 +02:00
|
|
|
if (ast.source.includes('\n') || ast.source.includes('\r')) {
|
|
|
|
reporter.error(
|
|
|
|
'Unexpected newline character(s) detected in SVG markup',
|
|
|
|
);
|
|
|
|
} else {
|
|
|
|
reporter.error(
|
|
|
|
'Unexpected character(s), most likely extraneous whitespace, detected in SVG markup',
|
|
|
|
);
|
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'negative-zeros';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Find negative zeros inside path
|
|
|
|
const negativeZeroMatches = Array.from(
|
|
|
|
iconPath.matchAll(negativeZerosRegexp),
|
|
|
|
);
|
|
|
|
if (negativeZeroMatches.length) {
|
|
|
|
// Calculate the index for each match in the file
|
|
|
|
const svgFileContent = $.html();
|
|
|
|
const pathDIndex = getPathDIndex(svgFileContent);
|
|
|
|
|
|
|
|
negativeZeroMatches.forEach((match) => {
|
|
|
|
const negativeZeroFileIndex = match.index + pathDIndex;
|
|
|
|
const previousChar = svgFileContent[negativeZeroFileIndex - 1];
|
|
|
|
const replacement = '0123456789'.includes(previousChar)
|
|
|
|
? ' 0'
|
|
|
|
: '0';
|
|
|
|
reporter.error(
|
|
|
|
`Found "-0" at index ${negativeZeroFileIndex} (should be "${replacement}")`,
|
|
|
|
);
|
|
|
|
});
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'icon-centered';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
if (!updateIgnoreFile && isIgnored(reporter.name, iconPath)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
const [minX, minY, maxX, maxY] = svgPathBbox(iconPath);
|
|
|
|
const targetCenter = iconSize / 2;
|
|
|
|
const centerX = +((minX + maxX) / 2).toFixed(iconFloatPrecision);
|
|
|
|
const devianceX = centerX - targetCenter;
|
|
|
|
const centerY = +((minY + maxY) / 2).toFixed(iconFloatPrecision);
|
|
|
|
const devianceY = centerY - targetCenter;
|
|
|
|
|
|
|
|
if (
|
|
|
|
Math.abs(devianceX) > iconTolerance ||
|
|
|
|
Math.abs(devianceY) > iconTolerance
|
|
|
|
) {
|
|
|
|
reporter.error(
|
|
|
|
`<path> must be centered at (${targetCenter}, ${targetCenter}); the center is currently (${centerX}, ${centerY})`,
|
|
|
|
);
|
|
|
|
if (updateIgnoreFile) {
|
|
|
|
ignoreIcon(reporter.name, iconPath, $);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'path-format';
|
|
|
|
|
|
|
|
const iconPath = $.find('path').attr('d');
|
|
|
|
|
|
|
|
const validPathFormatRegex = /^[Mm][MmZzLlHhVvCcSsQqTtAaEe0-9-,.\s]+$/;
|
|
|
|
if (!validPathFormatRegex.test(iconPath)) {
|
|
|
|
let errorMsg = 'Invalid path format',
|
|
|
|
reason;
|
|
|
|
|
|
|
|
if (!/^[Mm]/.test(iconPath)) {
|
|
|
|
// doesn't start with moveto
|
|
|
|
reason = `should start with \"moveto\" command (\"M\" or \"m\"), but starts with \"${iconPath[0]}\"`;
|
|
|
|
reporter.error(`${errorMsg}: ${reason}`);
|
|
|
|
}
|
|
|
|
|
|
|
|
const validPathCharacters = 'MmZzLlHhVvCcSsQqTtAaEe0123456789-,. ',
|
|
|
|
invalidCharactersMsgs = [],
|
|
|
|
pathDIndex = getPathDIndex($.html());
|
|
|
|
|
|
|
|
for (let [i, char] of Object.entries(iconPath)) {
|
|
|
|
if (validPathCharacters.indexOf(char) === -1) {
|
|
|
|
invalidCharactersMsgs.push(
|
|
|
|
`"${char}" at index ${pathDIndex + parseInt(i)}`,
|
|
|
|
);
|
2021-09-14 22:40:48 +02:00
|
|
|
}
|
2020-07-10 10:06:08 +02:00
|
|
|
}
|
2021-10-25 21:13:10 +02:00
|
|
|
|
|
|
|
// contains invalid characters
|
|
|
|
if (invalidCharactersMsgs.length > 0) {
|
|
|
|
reason = `unexpected character${
|
|
|
|
invalidCharactersMsgs.length > 1 ? 's' : ''
|
|
|
|
} found`;
|
|
|
|
reason += ` (${invalidCharactersMsgs.join(', ')})`;
|
|
|
|
reporter.error(`${errorMsg}: ${reason}`);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
},
|
|
|
|
(reporter, $, ast) => {
|
|
|
|
reporter.name = 'svg-format';
|
|
|
|
|
|
|
|
// Don't allow explicit '</path>' closing tag
|
|
|
|
if (ast.source.includes('</path>')) {
|
|
|
|
const reason =
|
|
|
|
`found a closing "path" tag at index ${ast.source.indexOf(
|
|
|
|
'</path>',
|
|
|
|
)}.` +
|
|
|
|
" The path should be self-closing, use '/>' instead of '></path>'.";
|
|
|
|
reporter.error(`Invalid SVG content format: ${reason}`);
|
|
|
|
}
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
2018-08-16 11:33:32 +02:00
|
|
|
};
|