Opened 4 years ago
Closed 3 years ago
#13577 closed bug (fixed)
New minimal selector engine fails on textNode objects
| Reported by: | albertcasademont@… | Owned by: | timmywil |
|---|---|---|---|
| Priority: | low | Milestone: | 2.0.1 |
| Component: | selector | Version: | 2.0b2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
textNode objects don't have the querySelectorAll function available. Maybe we could prevent querying on those types of nodes an avoid a js error?
Change History (4)
comment:1 Changed 4 years ago by timmywil
- Component changed from unfiled to selector
- Owner set to timmywil
- Priority changed from undecided to low
- Status changed from new to assigned
comment:2 Changed 3 years ago by dmethvin
- Milestone changed from None to 2.0
comment:3 Changed 3 years ago by dmethvin
- Milestone changed from 2.0 to 2.0.1
comment:4 Changed 3 years ago by Timmy Willison
- Resolution set to fixed
- Status changed from assigned to closed
Check nodeType of elements passed to selector-native's find. Fixes #13577.
Changeset: 8d3c0506c860e887d0005fd12da53f1d11ce2883
Note: See
TracTickets for help on using
tickets.
We'll just fail silently for text nodes.