UDOIT icon indicating copy to clipboard operation
UDOIT copied to clipboard

Add new check for empty tables

Open kennygperez opened this issue 5 years ago • 2 comments

Create an error for the case when the table tag has no tr's nested inside of it.

kennygperez avatar Feb 19 '20 17:02 kennygperez

Put in pr 22 for phpally to add this rule on that side. Once that is merged it will be necessary to add the rule on the UDOIT side and update the phpally version.

AlanFCMV avatar Dec 02 '21 15:12 AlanFCMV

PHPAlly has added this, but I can't seem to get it to fire correctly. I ran the raw HTML through the test suite in PHPAlly 1.1.1 and it caught it, so either we don't actually have that version (maybe the release package isn't actually the same as the changelog says?) or the issue isn't making its way to PHPAlly correctly.

Getting the Ufixit form working involves editing assets/js/Services/Ufixit.js to add a new component and editing a number of config files. It's also possible that the issue is just that I missed a config file somewhere. I recommend grepping to find where you need to edit config.

rob-3 avatar Apr 25 '22 14:04 rob-3