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.

Sign up for GitHub

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

Jump to bottom

"deltaE76 is not defined" #282

Closed
drewlarsen opened this issue Feb 8, 2023 · 5 comments
Closed

"deltaE76 is not defined" #282

drewlarsen opened this issue Feb 8, 2023 · 5 comments

Comments

@drewlarsen
Copy link

drewlarsen commented Feb 8, 2023 •
edited
Loading

Starting with an essentially empty html file:

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>colorio test</title>
    <script src="https://colorjs.io/dist/color.global.js"></script>
</head>
<body>
</body>
</html>

and following the instruction from the readme:

Or, if you'd rather just have Color as a global variable, the classic way, just include the following script in your HTML:

<script src="https://colorjs.io/dist/color.global.js"></script>

I get the following error in the browser console:

index.js:9 Uncaught ReferenceError: deltaE76 is not defined
    at index.js:9:2
    at index.js:32:30

image
@MeltonEngServ
Copy link

MeltonEngServ commented Feb 8, 2023 •
edited
Loading

I was using the library through a CDN as well, and It was working well, but now I get the same error. I think the problem comes from the last commit Allow to extend the deltaE methods #280

@isohelio
Copy link

isohelio commented Feb 8, 2023

Same here.

Is there any way to refer to the previous version, while waiting for a fix?

@isohelio
Copy link

isohelio commented Feb 8, 2023

It's breaking the colorjs demos, if you want to see an example.

https://colorjs.io/apps/picker/

@qgustavor
Copy link

Would not be better if colorjs.io handled versioning and the documentation recommended locking versions? For the while I'll be using https://unpkg.com/[email protected]/dist/color.js instead of https://colorjs.io/dist/color.js.

@LeaVerou LeaVerou mentioned this issue Feb 8, 2023
Merged
@svgeesus
Copy link
Member

Closed by #283

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@qgustavor @svgeesus @drewlarsen @isohelio @MeltonEngServ