WPIntell

Source evidence

Request: Please update JS code

Attendance Manager · support · 2015-10-16T16:57:00+00:00

complaintsentiment
highseverity
1.0relevance
3replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

7 / 29 rows with source links

24.1% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

22 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
xmmassage unresolved
Hello again ), Can you please update your front.js and change: FROM jQuery( document ).ready( function( $ ) { … }) TO document.addEventListener(‘DOMContentLoaded’, function(event) { … }) Many SEO optimization plugins move JS files in footer of the website. In this case JQUERY is available later then your script require it. There are several more issues why it is not good to be tied with JQUERY untill document loaded. Anyway, all users who is using such plugins will get “JQuerry is not defined” error in your script. Using simple JS is much better and risk free. Thank you! https://wordpress.org/plugins/attendance-manager/ Thank you for your feedback. I wanted to use jQuery. But a parameter to “wp_enqueue_script()” seems to have been insufficient. https://codex.wordpress.org/Function_Reference/wp_enqueue_script 3rd parameter: $deps The handle of all the registered scripts that this script depends on, that is the scripts that must be loaded before this script. 5th parameter: $in_footer When the theme has the wp_footer() template tag, the script is placed before the </body> end tag. I will correct these parameters. Thank you. Not sure I understood the problem you are trying to describe ). BTW: Can you please add some more information about your updates. It is second update with the comment “Some filters were added.”. Which filters, what for, where ))? You see, I’m using hardly hacked version of your plugin. Most of hacks are about UI and it is not really easy decision for me to update it, because I need to put my hacks back each time. So, I just want to know if new version worth updating or not ). PS: If you have time I can e-mail to you with the link to show you how and where I’m using your plugin and discuss my UI hacks. Maybe some of them can become optional in your plugin. I am depending on Google translation too much, because I am not good at English. So I want to apologize for lack of my understanding. Sorry.

Comments

3 shown
tnomi 2015-10-16T22:29:00+00:00

Thank you for your feedback. I wanted to use jQuery. But a parameter to “wp_enqueue_script()” seems to have been insufficient. https://codex.wordpress.org/Function_Reference/wp_enqueue_script 3rd parameter: $deps The handle of all the registered scripts that this script depends on, that is the scripts that must be loaded before this script. 5th parameter: $in_footer When the theme has the wp_footer() template tag, the script is placed before the </body> end tag. I will correct these parameters. Thank you.

xmmassage 2015-10-17T19:41:00+00:00

Not sure I understood the problem you are trying to describe ). BTW: Can you please add some more information about your updates. It is second update with the comment “Some filters were added.”. Which filters, what for, where ))? You see, I’m using hardly hacked version of your plugin. Most of hacks are about UI and it is not really easy decision for me to update it, because I need to put my hacks back each time. So, I just want to know if new version worth updating or not ). PS: If you have time I can e-mail to you with the link to show you how and where I’m using your plugin and discuss my UI hacks. Maybe some of them can become optional in your plugin.

tnomi 2015-10-18T15:40:00+00:00

I am depending on Google translation too much, because I am not good at English. So I want to apologize for lack of my understanding. Sorry.