Skip to content

Switch classList to use contains?#37962

Merged
Josh-Cena merged 1 commit intomdn:mainfrom
bkardell:patch-1
Feb 4, 2025
Merged

Switch classList to use contains?#37962
Josh-Cena merged 1 commit intomdn:mainfrom
bkardell:patch-1

Conversation

@bkardell
Copy link
Copy Markdown
Contributor

@bkardell bkardell commented Feb 4, 2025

Not sure the idea of using [0] index here but it seems better to demonstrate via contains() as the DOMTokenList API is pretty nice and not inherently subject to problems that indexing would be

Not sure the idea of using [0] index here but it seems better to demonstrate via contains() as the DOMTokenList API is pretty nice and not inherently subject to problems that indexing would be
@bkardell bkardell requested a review from a team as a code owner February 4, 2025 17:47
@bkardell bkardell requested review from estelle and removed request for a team February 4, 2025 17:47
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed labels Feb 4, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 4, 2025

Preview URLs

@Josh-Cena Josh-Cena merged commit 5c1847f into mdn:main Feb 4, 2025
@bkardell bkardell deleted the patch-1 branch February 18, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants