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

Async and Defer is broken #8

Open
Remzi1993 opened this issue May 31, 2019 · 4 comments
Open

Async and Defer is broken #8

Remzi1993 opened this issue May 31, 2019 · 4 comments
Assignees
Labels
Bug Something isn't working Help wanted Extra attention is needed Pinned

Comments

@Remzi1993
Copy link
Owner

Async and Defer are broken for both jQuery and jQuery Migrate.

@Remzi1993 Remzi1993 added Bug Something isn't working Help wanted Extra attention is needed labels May 31, 2019
@Remzi1993 Remzi1993 self-assigned this May 31, 2019
@stale
Copy link

stale bot commented Jul 30, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Won't fix This will not be worked on label Jul 30, 2019
@Remzi1993
Copy link
Owner Author

Will be fixed in the future. This has low priority.

@stale stale bot removed the Won't fix This will not be worked on label Jul 30, 2019
@stale stale bot added the Won't fix This will not be worked on label Sep 28, 2019
@Remzi1993 Remzi1993 added Pinned and removed Won't fix This will not be worked on labels Sep 28, 2019
Repository owner deleted a comment from stale bot Sep 29, 2019
@oldrup
Copy link

oldrup commented Aug 19, 2020

Hey! Thank you for creating this plugin, helps me use more modern and safe versions of jquery if at all needed on a site.

I'm not too much of the dysfunctional async/defer option though. Most caching plugins offers that functionality anyway, and as it isn't working, why not just remove the option completely? Would clean up the UI a bit.

Stay safe!
Bjarne

@cp-bwg
Copy link

cp-bwg commented May 7, 2021

Hey! Thank you for creating this plugin, helps me use more modern and safe versions of jquery if at all needed on a site.

I'm not too much of the dysfunctional async/defer option though. Most caching plugins offers that functionality anyway, and as it isn't working, why not just remove the option completely? Would clean up the UI a bit.

Stay safe!
Bjarne

Agreed - if something is not working at all - reference this issue as a future enhancement, but remove the options.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working Help wanted Extra attention is needed Pinned
Projects
None yet
Development

No branches or pull requests

3 participants