VelocityJS: Missing pattern: padding

[benzmuircroft]

Your system information

• VelocityJS version: https://cdn.jsdelivr.net/npm/[email protected]/velocity.min.js
• Browser: brave
• Operating System: Ubuntu Mate

Checklist

• Is this an issue with code?: [Yes/No]
• Is this an issue with documentation?: [Yes/No]
• Have you reproduced this in other browsers?: [Yes/No]
• Have you checked for updates?: [Yes/No] I found the cdn link from your git
• Have you checked for similar open issues?: [Yes/No]

$ev[j].velocity/*animate*/({
   'height': $ev[j].attr('h')+'px',
    'opacity': 1,
    'padding': '23px 8px '+(first_ev==1000?'0.08':'0')+'px',
    'margin-bottom': '4px'
},450,function(){...

[Rycochet]

Not a bug - that's you using it as a shortcut, and padding needs 4 values. You're passing 3 which the browser will auto-expand to four - so when it's finding the value it'll not match the three you have supplied. The solution (for now) is to either use force-feeding and supply the from value as 3 values too, or supply 4 values so the browser handles it all.

Saying that, going to leave this open, as I feel that this is a good one to have a handler for those shortcut values, and to auto-convert 1/2/3 into 4 values so it's all transparent (same for margin, border etc) :-)

[benzmuircroft]

yeah, good idea!