Unexpected unknown propertys in wp-includes/js/crop/cropper.css

[ozmatflc]

Line 88: wid\th: 6px;

Is the backslash intentional?

[ozmatflc]

Replying to ozmatflc:

Line 88: wid\th: 6px;

Is the backslash intentional?

Same in line 89: hei\ght: 6px;

[tobifjellner]

The comment in line 84 hints that this probably is intentional:

/* fix IE 5 box model */

[SergeyBiryukov]

Hi @ozmatflc, welcome to WordPress Trac! Thanks for the ticket.

Introduced in [4629], replaced with Jcrop in [10846], replaced with imgAreaSelect in [14907].

Yes, this appears to be a "CSS hack" for IE 5. While that browser is ​no longer supported, these older libraries are only kept for backward compatibility, I don't see a benefit in editing them.

[desrosj]

Thanks again for the ticket, @ozmatflc!

I agree with @SergeyBiryukov here, so I am going to close this out.

Hope to see more tickets from you in the near future!

[SIRavecavec]

Replying to SergeyBiryukov:

Hi @ozmatflc, welcome to WordPress Trac! Thanks for the ticket.

Introduced in [4629], replaced with Jcrop in [10846], replaced with imgAreaSelect in [14907].

Yes, this appears to be a "CSS hack" for IE 5. While that browser is ​no longer supported, these older libraries are only kept for backward compatibility, I don't see a benefit in editing them.

The benefits to edit and fix them, is code quality, passing security validation and internal audits!

I am still not sure why the OLD mentality if its not bothering me and is not burning, let it be.

Its easy to say wont be taken out for backward compatibilty, so lets start building software backwards not forward moving.

I was just about to open a ticket while this came out with many other on a sonarqube test done recently.

No matter what hack that is for what purpose, first of all has no meaning at all to keep it, second of all it invalidates a file that has entire rest of code clean up to that point.

So again what is with this approach that keeps core code bloated with errors same with jQuery and many other sides that have known vulnerabilities for years now and if a few of them arent properly exploited, let's not move on to fix them.