Can we nuke this?
console.log('SkipTo loaded...');
No idea where the source is coming from but its compiled into media/vendor/skipto/js/skipto.js and skipto.min.js
We must be modifying the source somehow as there is Joomla name spacing in it.
Nothing in console.log
unneeded debug information output
Labels |
Added:
?
|
We need to make a PR on their repo, again
oh does it have history then haha ? I saw the Joomla specific lines above the console.log so assumed you were modifying their code anyway and could easily nuke this ?
// Initialize skipto menu button with onload event
window.addEventListener('load', function() {
SkipTo.init(window.SkipToConfig ||
window.Wordpress ||
((typeof window.Joomla === 'object' && typeof window.Joomla.getOptions === 'function') ? window.Joomla.getOptions('skipto-settings', {}) : {})
);
console.log('SkipTo loaded...'); // jshint ignore:line
});
It would be a shame to leave this one, just as we get rid of the jQuery.Migrate console.log that has been following Joomla console logs around for the last decade ;-)
Yeah man, I added those: paypal/skipto#69
I'll try to nuke it but you know it's their call
Ta - will nuke this issue for now then as can't really do anything here.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-02-02 22:33:15 |
Closed_By | ⇒ | PhilETaylor |
For reference paypal/skipto#79
Thanks.
@PhilETaylor paypal/skipto#79 was merged so now we have to wait for their next release
I will push a new release shortly as there are some other bugs fixed upstream
@brianteeman npm says that last release was 19 days ago
@dgrammatiko ? maybe?