search

BuilderIO / builder

Drag and drop headless CMS for React, Vue, Svelte, Qwik, and more
https://builder.io
MIT License
6.97k stars 860 forks source link

Qwik Builder useVisibleTask default #2345

Open ekwoka opened 1 year ago

ekwoka commented 1 year ago

Describe the bug a fresh build loads js from a useVisibleTask on the builder content element. the builder.io site does not have this but it seems to be baked into how Builder works. Why is it different? How can I prevent loading the extra code that seems unecessary?

To Reproduce Steps to reproduce the behavior:

  1. make new qwik with builder integration
  2. build and serve
  3. see js loads

Expected behavior No JS loads since it isn't doing anything (yet) like the builder.io site.

divyeesh-40023 commented 1 year ago

Respected sir

On Thu, 15 Jun, 2023, 10:51 am Eric Kwoka, @.***> wrote:

Describe the bug a fresh build loads js from a useVisibleTask on the builder content element. the builder.io site does not have this but it seems to be baked into how Builder works. Why is it different? How can I prevent loading the extra code that

Thank you for your support

seems unecessary?

To Reproduce Steps to reproduce the behavior:

  1. make new qwik with builder integration
  2. build and serve
  3. see js loads

Expected behavior No JS loads since it isn't doing anything (yet) like the builder.io site.

— Reply to this email directly, view it on GitHub https://github.com/BuilderIO/builder/issues/2345, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5Z56XUFTVWUOBISNRNLG2LXLKLXFANCNFSM6AAAAAAZHJOPI4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

manucorporat commented 1 year ago

I will work to optimize the qwik SDK! it should deliver zero js!