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

Original critters options don't have any impact #67

Closed
drumsta opened this issue Feb 9, 2023 · 4 comments
Closed

Original critters options don't have any impact #67

drumsta opened this issue Feb 9, 2023 · 4 comments

Comments

@drumsta
Copy link

drumsta commented Feb 9, 2023

Hello,

I'm trying to fine tune critters process, but it seems options from original Critters don't have any impact at all.

The configuration file looks like below:

  integrations: [
    critters({
      minimumExternalSize: 1000000,
      keyframes: "all",
      pruneSource: true,
      noscriptFallback: false,
    }),
  ],
});

No one option listed above doesn't have any impact on the final build - builds look identically with or without these options.

Any ideas?

@NikolaRHristov
Copy link
Member

NikolaRHristov commented Feb 10, 2023

pruneSource doesn't work at all. This has been referenced before (discussed offline) #1 and there's a new issue about it - GoogleChromeLabs/critters#111

I have created a new library (forked) that will fix this https://github.com/nikolaxhristov/critters

I don't know about the other options. You can try to manually invoke critters like in GoogleChromeLabs/critters#111 to see if your changes have any effect at all.

@drumsta
Copy link
Author

drumsta commented Feb 10, 2023

Thanks Nikola for your reply!

Are you going eventually to replace the dependency used in astro-critters to your new library mention in the answer above? Or will astro-critters still use orginal Chrome Critters under the hood?

I'm Astro user so that's why I'm interested.

Regards,

@NikolaRHristov
Copy link
Member

Yup, that's the plan @drumsta

@drumsta drumsta closed this as completed Feb 11, 2023
@drumsta
Copy link
Author

drumsta commented Feb 11, 2023

Thanks Nikola for clarofication.

All the best in the development!

I'm closing the issue.

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

No branches or pull requests

2 participants