Skip to content

Replace addClass with classList.add#6229

Merged
kevin-brown merged 1 commit into
select2:developfrom
p8:remove-jquery/addClass
Mar 2, 2023
Merged

Replace addClass with classList.add#6229
kevin-brown merged 1 commit into
select2:developfrom
p8:remove-jquery/addClass

Conversation

@p8

@p8 p8 commented Feb 20, 2023

Copy link
Copy Markdown
Contributor

This pull request includes a

  • Bug fix
  • New feature
  • Translation

The following changes were made

  • This replaces jQuery's addClass function with the vanilla JavaScript classList.add.

This is related to: #6196

@p8 p8 force-pushed the remove-jquery/addClass branch from d3b48bc to 7a8cffc Compare February 21, 2023 09:40
@kevin-brown

kevin-brown commented Feb 21, 2023

Copy link
Copy Markdown
Member

Can you rebase this with the latest changes on develop (I merged in your PR to fix the CI, thank you!) and back out the dist changes for me? The dist changes cause annoying merge conflicts across PRs so I prefer to recompile it separately after I merge batches of changes.

This replaces jQuery's `addClass` function with the vanilla JavaScript
`classList.add`.
@p8 p8 force-pushed the remove-jquery/addClass branch from 7a8cffc to ef3fc97 Compare February 21, 2023 20:36
@p8

p8 commented Mar 2, 2023

Copy link
Copy Markdown
Contributor Author

@kevin-brown I've rebased and removed the dist changes.

@kevin-brown kevin-brown merged commit 8bdafc3 into select2:develop Mar 2, 2023
arkonan added a commit to modolabs/select2 that referenced this pull request May 19, 2023
kevin-brown added a commit that referenced this pull request May 26, 2026
- Remove '(unreleased)' marker from the 4.1.0 heading
- Add new features: jQuery 4.0.0 support (#6332), originalEvent in
  close trigger args (#6079)
- Add bug fixes: placeholder misalignment (#6277), RTL choice remove
  button (#6257), digit-only data-placeholder (#6297), AJAX unselection
  with non-string IDs (#6241, #6335), optgroup child string coercion (#6338)
- Add translations: lb (#6131), ug (#6166), ar (#6175), zh-TW (#6157),
  id (#6153), tr (#6123), ro (#6190), de/es/fr/pt/pt-BR (#6132),
  pl (#6097, #6377), nb (#6213), fa (#6258), nl (#6269), pt-BR typo
  (#6200), missing bs/ca/da/fi strings (#6305)
- Add miscellaneous: native DOM replacements for jQuery attr/removeAttr
  (#6227, #6228), classList.add (#6229), jQuery removal from Utils and
  Translation (#6233), prop() removal (#6289), NPM trusted publishing
  (#6405)
maxwellfet928 pushed a commit to maxwellfet928/select2 that referenced this pull request Jun 8, 2026
This replaces jQuery's `addClass` function with the vanilla JavaScript
`classList.add`.
maxwellfet928 pushed a commit to maxwellfet928/select2 that referenced this pull request Jun 8, 2026
- Remove '(unreleased)' marker from the 4.1.0 heading
- Add new features: jQuery 4.0.0 support (select2#6332), originalEvent in
  close trigger args (select2#6079)
- Add bug fixes: placeholder misalignment (select2#6277), RTL choice remove
  button (select2#6257), digit-only data-placeholder (select2#6297), AJAX unselection
  with non-string IDs (select2#6241, select2#6335), optgroup child string coercion (select2#6338)
- Add translations: lb (select2#6131), ug (select2#6166), ar (select2#6175), zh-TW (select2#6157),
  id (select2#6153), tr (select2#6123), ro (select2#6190), de/es/fr/pt/pt-BR (select2#6132),
  pl (select2#6097, select2#6377), nb (select2#6213), fa (select2#6258), nl (select2#6269), pt-BR typo
  (select2#6200), missing bs/ca/da/fi strings (select2#6305)
- Add miscellaneous: native DOM replacements for jQuery attr/removeAttr
  (select2#6227, select2#6228), classList.add (select2#6229), jQuery removal from Utils and
  Translation (select2#6233), prop() removal (select2#6289), NPM trusted publishing
  (select2#6405)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants