-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add options for defer async etc. #5
Comments
What do you think would be the better approach:
I'm currently leaning more towards option 2. It could be used like this: <script src="<?php echo mixPath('/main.js') ?>" async> What's your opinion on this? |
I much prefer option 2 as well. Makes things less abstract. |
I tend to option 1 making the function behave like kirby's |
Thanks, @arnoson, didn’t think of that. |
Maybe you should add
$options = []
to the mix function? I can make PRThe text was updated successfully, but these errors were encountered: