You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Herve Boutemy (Jira)" <ji...@apache.org> on 2021/11/27 17:23:00 UTC
[jira] [Updated] (MSKINS-168) Switch Minification Engine from YUI to CLOSURE
[ https://issues.apache.org/jira/browse/MSKINS-168?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Herve Boutemy updated MSKINS-168:
---------------------------------
Fix Version/s: fuildo-1.10
(was: fluido-2.0)
> Switch Minification Engine from YUI to CLOSURE
> ----------------------------------------------
>
> Key: MSKINS-168
> URL: https://issues.apache.org/jira/browse/MSKINS-168
> Project: Maven Skins
> Issue Type: Improvement
> Components: Fluido Skin
> Affects Versions: fluido-1.9
> Reporter: Bakary DJIBA
> Assignee: Herve Boutemy
> Priority: Major
> Labels: anchorjs, bootstrap, js, jsEngine, minification
> Fix For: fuildo-1.10
>
>
> While working on Issues to add deep anchors(MSKINS-167) and Upgrade to Bootstrap 4 (MSKINS-97) the default jsEngine used by [minify-maven-plugin|https://samaxes.github.io/minify-maven-plugin/] raises issues during minify compile's step.
> Below the affected js libs:
> [bootstrap upgrade]
> + boostrap.bundle-4.4.1.js,
> + jquery-3.4.1.js
> [deep anchor features]
> + anchor-4.2.2.js
> I managed to modify anchor source file to workaround YUI issue (by replacing {{opts.class}} with {{opts['class']}}, seems to be a known limitation of YUI https://github.com/yui/yuicompressor/issues/77), but it will be hard to maintain, and definitely not an option for bootstrap upgrade
> These problems are fixed by a switch to CLOSURE engine.
> Therefore a this improvement seems useful or mandatory for Upgrading Bootstrap to version 4. Also to avoid modification in anchor js lib.
>
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)