Skip to content
This repository has been archived by the owner on May 17, 2022. It is now read-only.

Remove jquery #17

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Remove jquery #17

wants to merge 2 commits into from

Conversation

sapkra
Copy link
Member

@sapkra sapkra commented Jan 9, 2022

React Native might be broken! Since we removed jQuery, we accidentally introduced a dependency on the browser environment. I'm open for solutions.

A solution might be xmldom or svgdom.

sapkra added 2 commits January 9, 2022 01:41
- prefix xmlns to be usable with querySelector
@Fuzzyma
Copy link

Fuzzyma commented Feb 21, 2022

What a coincident. I am the maintainer of svgdom :D. But you probably wanna look at jsdom. It mirrors the browser environment pretty closely. svgdom is mostly useful if you need a bounding box or matrix calculations for your svgs. Well, I guess for this usecase it would work fine as well

@sapkra
Copy link
Member Author

sapkra commented Feb 22, 2022

It's not really a coincident. The PR is based on your PR and we already discussed the use of other libraries like svgdom in the comments.

@Fuzzyma
Copy link

Fuzzyma commented Feb 23, 2022

I guess it's too long ago to remember 😂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants