search

SSAgov / ANDI

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

$("#ANDI508-viewIframeList-button").click(function() is organized the opposite way as other buttons #202

Closed ollie-iterators closed 1 year ago

ollie-iterators commented 1 year ago

The view iFrame List button is organized oppositely than the other buttons in the ANDI module.

The first if statement for the iANDI button is : "if($(this).attr("aria-expanded") === "true")", while the first if statement for the lANDI button is: "if($(this).attr("aria-expanded") === "false").

I think it would be better if the buttons were coded the same way.

ollie-iterators commented 1 year ago

The same thing is true for the outline button for the headings list

JohnCotterSSA commented 1 year ago

Thank you for the review.