diff --git a/plugins/removeHiddenElems.js b/plugins/removeHiddenElems.js index efa1be43..51356835 100644 --- a/plugins/removeHiddenElems.js +++ b/plugins/removeHiddenElems.js @@ -72,7 +72,9 @@ exports.fn = function (item, params) { params.displayNone && computedStyle.display && computedStyle.display.type === 'static' && - computedStyle.display.value === 'none' + computedStyle.display.value === 'none' && + // markers with display: none still rendered + item.isElem('marker') === false ) { return false; } diff --git a/test/regression.js b/test/regression.js index 13de8c61..8cc5c49e 100644 --- a/test/regression.js +++ b/test/regression.js @@ -97,23 +97,26 @@ const runTests = async ({ svgFiles }) => { name.startsWith('animate-') || // breaks because of optimisation despite of script name === 'interact-pointer-04-f' || - // other cases which require complex changes - name === 'painting-marker-07-f' || + // messed gradients name === 'pservers-grad-18-b' || + // animated filter + name === 'filters-light-04-f' || + // removing wrapping breaks :first-child pseudo-class + name === 'styling-pres-04-f' || + // messed case insensitivity while inlining styles + name === 'styling-css-10-f' || + // rect is converted to path which matches wrong styles + name === 'styling-css-08-f' || + // other cases which require complex changes name === 'struct-image-02-b' || name === 'struct-use-10-f' || name === 'struct-use-11-f' || name === 'styling-css-01-b' || name === 'styling-css-03-b' || name === 'styling-css-04-f' || - name === 'styling-css-08-f' || - // unstable test - name === 'filters-light-04-f' || // mismatched draft cases name === 'struct-cond-overview-03-f' || - name === 'struct-use-07-b' || - name === 'styling-css-10-f' || - name === 'styling-pres-04-f' + name === 'struct-use-07-b' ) { console.info(`${name} is skipped`); skipped += 1;