medialize / ally.js

JavaScript library to help modern web applications with accessibility concerns
http://allyjs.io/
MIT License
1.53k stars 83 forks source link

ally.query.tabsequence possible string comparison of tabindex #160

Open paulie4 opened 6 years ago

paulie4 commented 6 years ago

When I edited your example in https://allyjs.io/api/query/tabsequence.html to include tabindex="10" and tabindex="11", ally.query.tabsequence is returning them before the tabindex="2", which makes me think that the code is possibly doing a string comparison instead of a numeric one. Here's an edited jsbin: https://jsbin.com/pesawikine/edit?html,js,output