Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace attr with getAttribute and setAttribute #6227

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

p8
Copy link
Contributor

@p8 p8 commented Feb 20, 2023

This pull request includes a

  • Bug fix
  • New feature
  • Translation

The following changes were made

  • This replaces jQuery's attr function with the vanilla JavaScript getAttribute and setAttribute.

This is related to: #6196

@p8 p8 force-pushed the remove-jquery/attr branch 3 times, most recently from 2e974ac to 6a56338 Compare February 21, 2023 18:43
@kevin-brown
Copy link
Member

kevin-brown commented Feb 21, 2023

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.

@p8
Copy link
Contributor Author

p8 commented Feb 21, 2023

@kevin-brown Thanks! Sure will do.

@p8
Copy link
Contributor Author

p8 commented Mar 2, 2023

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

This replaces jQuery's `attr` function with the vanilla JavaScript
`getAttribute` and `setAttribute`.
This replaces jQuery's `attr` function with the vanilla JavaScript
`getAttribute` and `setAttribute`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants