SSAgov / ANDI

A tool to test web content for accessibility and 508 compliance.
https://www.ssa.gov/accessibility/andi/help/howtouse.html
Other
293 stars 75 forks source link

Code may be unnecessary in hANDI #140

Closed ollie-iterators closed 1 year ago

ollie-iterators commented 2 years ago

The code: $(element).css("text-indent") != "0" || $(element).css("text-indent") != "0px" on line 72 of hANDI.js may be unnecessary because in the same if statement, it is checked if "text-indent" is less than -998.

If the condition with parseInt is True, then that means that the other conditions are also true. And if the value of parseInt goes to 0, then it should not be less than -998.

JohnCotterSSA commented 1 year ago

Good catch. Fix incoming