Hi, Now that js/update-location.js is converted to jQuery -- thanks Michal -- I believe we could remove the loading of mootools.js from index.php.
Unless there is some dependency to Mootools inside js/common.js but I think not.
Hi
Dne Thu, 01 Apr 2010 11:03:00 -0400 Marc Delisle marc@infomarc.info napsal(a):
Now that js/update-location.js is converted to jQuery -- thanks Michal -- I believe we could remove the loading of mootools.js from index.php.
Unless there is some dependency to Mootools inside js/common.js but I think not.
Probably yes.
BTW: Does anybody know where all do we use mootools?
Michal Čihař a écrit :
Hi
Dne Thu, 01 Apr 2010 11:03:00 -0400 Marc Delisle marc@infomarc.info napsal(a):
Now that js/update-location.js is converted to jQuery -- thanks Michal -- I believe we could remove the loading of mootools.js from index.php.
Unless there is some dependency to Mootools inside js/common.js but I think not.
Probably yes.
Yes there is a dependency?
BTW: Does anybody know where all do we use mootools?
I expect that it's only where we include it. Is it oversimplified?
Hi
Dne Thu, 01 Apr 2010 11:20:30 -0400 Marc Delisle marc@infomarc.info napsal(a):
Michal Čihař a écrit :
Hi
Dne Thu, 01 Apr 2010 11:03:00 -0400 Marc Delisle marc@infomarc.info napsal(a):
Now that js/update-location.js is converted to jQuery -- thanks Michal -- I believe we could remove the loading of mootools.js from index.php.
Unless there is some dependency to Mootools inside js/common.js but I think not.
Probably yes.
Yes there is a dependency?
Yes, we can remove it.
BTW: Does anybody know where all do we use mootools?
I expect that it's only where we include it. Is it oversimplified?
The only mootools user I found is mootools_common.js (it should be rather named replication.js), but I checked only separate js files and not inline code.
Hi
Dne Thu, 1 Apr 2010 17:33:12 +0200 Michal Čihař michal@cihar.com napsal(a):
The only mootools user I found is mootools_common.js (it should be rather named replication.js), but I checked only separate js files and not inline code.
Ah okay, in functions.js there are some blob streaming functions which do also use mootools.
Michal Čihař a écrit :
Hi
Dne Thu, 1 Apr 2010 17:33:12 +0200 Michal Čihař michal@cihar.com napsal(a):
The only mootools user I found is mootools_common.js (it should be rather named replication.js), but I checked only separate js files and not inline code.
Ah okay, in functions.js there are some blob streaming functions which do also use mootools.
Mootools also used in /setup.
In infinite wisdom Michal Čihař michal@cihar.com wrote:
BTW: Does anybody know where all do we use mootools?
One way to find this would be to run the selenium test scripts and then grep the apache logs for mootool*.js. The referrer page in the access log should give an idea of which all pages use mootools (or include it).
Hi
Dne Thu, 01 Apr 2010 09:37:00 -0700 Raj Shekhar rajlist@rajshekhar.net napsal(a):
In infinite wisdom Michal Čihař michal@cihar.com wrote:
BTW: Does anybody know where all do we use mootools?
One way to find this would be to run the selenium test scripts and then grep the apache logs for mootool*.js. The referrer page in the access log should give an idea of which all pages use mootools (or include it).
We don't have problem finding code which references to mootools*js, but rather code which uses some of mootools functions.
Just a related issue - main.php loads jQuery two times, it's added to js_include in common.inc.php and main.php
2010/4/1 Marc Delisle marc@infomarc.info:
Hi, Now that js/update-location.js is converted to jQuery -- thanks Michal -- I believe we could remove the loading of mootools.js from index.php.
Unless there is some dependency to Mootools inside js/common.js but I think not.
-- Marc Delisle http://infomarc.info
Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
Dne Thu, 1 Apr 2010 17:08:43 +0200 Piotr Przybylski piotr.prz@gmail.com napsal(a):
Just a related issue - main.php loads jQuery two times, it's added to js_include in common.inc.php and main.php
It was the case in more files, fixed in Git.
Michal Čihař a écrit :
Dne Thu, 1 Apr 2010 17:08:43 +0200 Piotr Przybylski piotr.prz@gmail.com napsal(a):
Just a related issue - main.php loads jQuery two times, it's added to js_include in common.inc.php and main.php
It was the case in more files, fixed in Git.
Ah, I was not aware that common.inc.php now loads it. Well, this brings another question: how will we document which code depends on jQuery? or do we need to document it?
Hi
Dne Thu, 01 Apr 2010 11:28:11 -0400 Marc Delisle marc@infomarc.info napsal(a):
Ah, I was not aware that common.inc.php now loads it. Well, this brings another question: how will we document which code depends on jQuery? or do we need to document it?
As the location updater requires it and it is used on each page, I think it's safe to assume jQuery is always present.
And it was probably mine fault, because other users were added earlier than the one in common and I should have removed it.
Piotr Przybylski a écrit :
Just a related issue - main.php loads jQuery two times, it's added to js_include in common.inc.php and main.php
Thanks I'll fix this.
-
Marc Delisle -
Michal Čihař -
Piotr Przybylski -
Raj Shekhar