You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
hasAria() currently delegates to hasAttribute(), but the arguments.length check in hasAttribute() does not work correctly, which makes hasAria('disabled') crash.
The text was updated successfully, but these errors were encountered:
As "just" applying aria-disabled="true/false" has no real effect without the accompanying JS / CSS — clicking an <input type="checkbox" aria-disabled="true/false"> toggles just fine — I would understand, if this addition is not desirable.
good question... I guess right now we're a bit inconsistent in that regard 😞
what I'm wondering is if we should have something like .isChecked({ withAriaSupport: true | false }) and then potentially also a global setting like setup(QUnit.assert, { withAriaSupport: true | false }) 🤔
buschtoens
added a commit
to buschtoens/qunit-dom
that referenced
this issue
Jul 29, 2021
hasAria()
currently delegates tohasAttribute()
, but thearguments.length
check inhasAttribute()
does not work correctly, which makeshasAria('disabled')
crash.The text was updated successfully, but these errors were encountered: