jquery / jquery/sizzle

Certain attributes should be treated as case-insensitive when comparing them

Open
#122 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

3 - Low Has a jQuery counterpart needsreview
Dominant language
JavaScript
Stars
6.3k
Forks
927
PR merge metrics
No merged PRs in 30d

Description

Ported from jQuery bug http://bugs.jquery.com/ticket/12024

http://jsfiddle.net/timmywil/n8zDu/14/

The inputs in the example all have uppercase values for their attributes while the value in the selector is lowercase. When :not contains a complex selector, querySelectorAll fails and Sizzle handles the attribute comparisons but does not account for these attributes being case-insensitive. We can fix that, but it will probably require a list within Sizzle specifying which attributes are case-insensitive in both HTML and XML.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the linked jsFiddle and trace the Sizzle attribute-comparison path used when :not contains a complex selector and querySelectorAll fails. Identify which attributes require case-insensitive comparison in HTML and XML, then verify that the example's uppercase attribute values match lowercase selector values without changing other comparisons.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.