Is this set of code outdated?

FluidSxul·9/27/2024 in JS Development

Was trying to implement something that I saw in another wiki on a wiki that I'm an admin on, and well, trying to test it doesn't seem to be working.

// Checks if a div with the id "spoilers_image" is clicked and then adds a
// "display:none" style attribute to a div with the id "spoilers".

document.getElementById("spoilers_image").addEventListener("click", function() {
    $('#spoilers').fadeOut(400, function () {
        this.style.display = "none";
    });
});

Source: The DDLC wiki.

KockaAdmiralac·9/27/2024

That seems correct. Where are you testing it?

FluidSxul·9/27/2024

I've placed it in the wiki's Common.js page.

Yet, when I create a template, and use the div and image with the ids as shown, it doesn't seem to do what it's supposed to do

KockaAdmiralac·9/29/2024

What is your wiki?

FluidSxul·9/29/2024

CONTINUED: The Sonic Oddities wiki.

I THINK i might've figured out why, i just need to wait a bit for it to apply.

KockaAdmiralac·10/1/2024

You've got to remove the broken code above it:

/* Username replace feature
 * Inserts viewing user's name into <span class="insertusername"></span>
 * Put text inside the spans to be viewed by logged out users
 * Originally by [[Wikia:User:Splarka|Splarka]], then by [[User:Spang|Spang]],
 * This (jQuery) version by [[wikia:User:Joeyaa]], written to be backwards compatible
 */
 
if (wgUserName != 'null') {
	$('.insertusername').html(wgUserName);
}

What do you think?