cssmin icon indicating copy to clipboard operation
cssmin copied to clipboard

Fixed the spaces issue with calc() and sum operation

Open oscarmlage opened this issue 7 years ago • 3 comments

If you have some css code like:

calc(50px + 2%)

The cssmin should keep the spaces around the plus signal because otherwhise (50px+2%) is not w3c valid.

oscarmlage avatar Mar 12 '18 10:03 oscarmlage

Hey @zacharyvoase, I've read in the project description that it's no longer maintained. I don't know the real status but I would like to continue using it for my projects.

The question from my side is, what do you think it's better for this kind of improvements, just add a PR like I did this time or fork the project and release it in pip with different name (respecting authors, contributors, license, etc...)? (said pip because it's easier for environments installations, etc...) .

Thanks in advance and sorry about the inconvenience it may cause.

oscarmlage avatar Mar 12 '18 10:03 oscarmlage

For everyone else having this issue a workaround it to use:

calc(50px - 2% * -1)

Lauszus avatar Dec 06 '18 22:12 Lauszus

@oscarmlage If you could for and make your own repo for this, that would be great. I don't own this but a maintained minifier would be nice.

UCYT5040 avatar Apr 28 '22 20:04 UCYT5040