Skip to content

Use the URL api rather than normalize-url#3614

Open
jcoyne wants to merge 1 commit intomainfrom
url-api
Open

Use the URL api rather than normalize-url#3614
jcoyne wants to merge 1 commit intomainfrom
url-api

Conversation

@jcoyne
Copy link
Copy Markdown
Collaborator

@jcoyne jcoyne commented Jan 24, 2023

The URL API is now in all browsers that we support: https://caniuse.com/url

@jcoyne jcoyne force-pushed the url-api branch 2 times, most recently from 98990e1 to b607f9b Compare January 25, 2023 14:23
@cbeer
Copy link
Copy Markdown
Collaborator

cbeer commented Jan 25, 2023

I think normalize-url provides additional functionality to deal with slightly malformed IIIF manifest data, doesn't it? I don't think it's completely equivalent to URL.

@jcoyne
Copy link
Copy Markdown
Collaborator Author

jcoyne commented Jan 25, 2023

@cbeer they are not equivalent, but I think that URI handles the cases we actually care about. We do have some bad tests though that rely on this difference (#3620 for example)

The URL API is now in all browsers that we support: https://caniuse.com/url
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