fixes some bad html (#27481)

code/controllers/subsystem/SSevents.dm

@@ -11,11 +11,11 @@ SUBSYSTEM_DEF(events)
 	// UI vars
 	var/window_x = 700
 	var/window_y = 600
-	var/table_options = " align='center'"
-	var/head_options = " style='font-weight:bold;'"
-	var/row_options1 = " width='85px'"
-	var/row_options2 = " width='260px'"
-	var/row_options3 = " width='150px'"
+	var/table_options = "align='center'"
+	var/head_options = "style='font-weight:bold;'"
+	var/row_options1 = "width='85px'"
+	var/row_options2 = "width='260px'"
+	var/row_options3 = "width='150px'"
 
 	// Event vars
 	var/datum/event_container/selected_event_container = null
@@ -109,8 +109,8 @@ SUBSYSTEM_DEF(events)
 		html += "Time till start: [round(event_time / 600, 0.1)]<br>"
 		html += "<div class='block'>"
 		html += "<h2>Available [GLOB.severity_to_string[selected_event_container.severity]] Events (queued & running events will not be displayed)</h2>"
-		html += "<table[table_options]>"
-		html += "<tr[head_options]><td[row_options2]>Name </td><td>Weight </td><td>MinWeight </td><td>MaxWeight </td><td>OneShot </td><td>Enabled </td><td><span class='alert'>CurrWeight </span></td><td>Remove</td></tr>"
+		html += "<table [table_options]>"
+		html += "<tr [head_options]><td [row_options2]>Name </td><td>Weight </td><td>MinWeight </td><td>MaxWeight </td><td>OneShot </td><td>Enabled </td><td><span class='alert'>CurrWeight </span></td><td>Remove</td></tr>"
 		for(var/datum/event_meta/EM in selected_event_container.available_events)
 			html += "<tr>"
 			html += "<td>[EM.name]</td>"
@@ -127,8 +127,8 @@ SUBSYSTEM_DEF(events)
 
 		html += "<div class='block'>"
 		html += "<h2>Add Event</h2>"
-		html += "<table[table_options]>"
-		html += "<tr [head_options]><td[row_options2]>Name</td><td[row_options2]>Type</td><td[row_options1]>Weight</td><td[row_options1]>OneShot</td></tr>"
+		html += "<table [table_options]>"
+		html += "<tr [head_options]><td [row_options2]>Name</td><td [row_options2]>Type</td><td [row_options1]>Weight</td><td [row_options1]>OneShot</td></tr>"
 		html += "<tr>"
 		html += "<td><A align='right' href='byond://?src=[UID()];set_name=\ref[new_event]'>[new_event.name ? new_event.name : "Enter Event"]</A></td>"
 		html += "<td><A align='right' href='byond://?src=[UID()];set_type=\ref[new_event]'>[new_event.event_type ? new_event.event_type : "Select Type"]</A></td>"
@@ -143,8 +143,8 @@ SUBSYSTEM_DEF(events)
 		html += "<div class='block'>"
 		html += "<h2>Event Start</h2>"
 
-		html += "<table[table_options]>"
-		html += "<tr[head_options]><td[row_options1]>Severity</td><td[row_options1]>Starts At</td><td[row_options1]>Starts In</td><td[row_options3]>Adjust Start</td><td[row_options1]>Pause</td><td[row_options1]>Interval Mod</td></tr>"
+		html += "<table [table_options]>"
+		html += "<tr [head_options]><td [row_options1]>Severity</td><td [row_options1]>Starts At</td><td [row_options1]>Starts In</td><td [row_options3]>Adjust Start</td><td [row_options1]>Pause</td><td [row_options1]>Interval Mod</td></tr>"
 		for(var/severity = EVENT_LEVEL_MUNDANE to EVENT_LEVEL_MAJOR)
 			var/datum/event_container/EC = event_containers[severity]
 			var/next_event_at = max(0, EC.next_event_time - world.time)
@@ -170,8 +170,8 @@ SUBSYSTEM_DEF(events)
 
 		html += "<div class='block'>"
 		html += "<h2>Next Event</h2>"
-		html += "<table[table_options]>"
-		html += "<tr[head_options]><td[row_options1]>Severity</td><td[row_options2]>Name</td><td[row_options3]>Event Rotation</td><td>Clear</td></tr>"
+		html += "<table [table_options]>"
+		html += "<tr [head_options]><td [row_options1]>Severity</td><td [row_options2]>Name</td><td [row_options3]>Event Rotation</td><td>Clear</td></tr>"
 		for(var/severity = EVENT_LEVEL_MUNDANE to EVENT_LEVEL_MAJOR)
 			var/datum/event_container/EC = event_containers[severity]
 			var/datum/event_meta/EM = EC.next_event
@@ -187,8 +187,8 @@ SUBSYSTEM_DEF(events)
 		html += "<div class='block'>"
 		html += "<h2>Running Events</h2>"
 		html += "Estimated times, affected by master controller delays."
-		html += "<table[table_options]>"
-		html += "<tr[head_options]><td[row_options1]>Severity</td><td[row_options2]>Name</td><td[row_options1]>Ends At</td><td[row_options1]>Ends In</td><td[row_options3]>Stop</td></tr>"
+		html += "<table [table_options]>"
+		html += "<tr [head_options]><td [row_options1]>Severity</td><td [row_options2]>Name</td><td [row_options1]>Ends At</td><td [row_options1]>Ends In</td><td [row_options3]>Stop</td></tr>"
 		for(var/datum/event/E in active_events)
 			if(!E.event_meta)
 				continue

code/datums/components/riding/riding_vehicle.dm

@@ -116,7 +116,7 @@
 
 /datum/component/riding/vehicle/scooter/skateboard/proc/on_examine(datum/source, mob/user, list/examine_list)
 	SIGNAL_HANDLER //COMSIG_PARENT_EXAMINE
-	examine_list += "<span class='notice>Going nice and slow at walk speed will prevent crashing into things.</span>"
+	examine_list += "<span class='notice'>Going nice and slow at walk speed will prevent crashing into things.</span>"
 
 /datum/component/riding/vehicle/scooter/skateboard/vehicle_mob_buckle(datum/source, mob/living/rider, force = FALSE)
 	. = ..()

code/game/gamemodes/cult/runes.dm

@@ -592,7 +592,7 @@ structure_check() searches for nearby cultist structures required for the invoca
 	. = ..()
 	if(IS_CULTIST(user) || user.stat == DEAD)
 		. += "<b>Sacrifices unrewarded:</b><span class='cultitalic'> [length(GLOB.sacrificed) - sacrifices_used]</span>"
-		. += "<b>Sacrifice cost per ressurection:</b><span class='cultitalic> [SOULS_TO_REVIVE]</span>"
+		. += "<b>Sacrifice cost per ressurection:</b><span class='cultitalic'> [SOULS_TO_REVIVE]</span>"
 
 /obj/effect/rune/raise_dead/proc/revive_alive(mob/living/target)
 	target.visible_message("<span class='warning'>Dark magic begins to surround [target], regenerating their body.</span>")

code/game/gamemodes/nuclear/nuclearbomb.dm

@@ -365,7 +365,7 @@ GLOBAL_VAR(bomb_set)
 		if(!I.use_tool(src, user, 40, 5, volume = I.tool_volume) || removal_stage != NUKE_COVER_OPEN)
 			return
 		visible_message("<span class='notice'>[user] cuts apart the anchoring system sealant on [src].</span>",\
-		"<span class='notice'>You cut apart the anchoring system's sealant.</span></span>")
+		"<span class='notice'>You cut apart the anchoring system's sealant.</span>")
 		removal_stage = NUKE_SEALANT_OPEN
 	update_icon(UPDATE_OVERLAYS)
 

code/game/gamemodes/wizard/magic_tarot.dm

@@ -122,7 +122,7 @@
 		user.drop_item()
 		var/obj/item/magic_tarot_card/MTC = new /obj/item/magic_tarot_card(get_turf(src), null, tarot_type)
 		user.put_in_hands(MTC)
-		to_chat(user, "</span><span class='hierophant'>You put your Vision into [src], and your Vision makes a work of Art! [MTC.name]... [MTC.card_desc]</span>") //No period on purpose.
+		to_chat(user, "<span class='hierophant'>You put your Vision into [src], and your Vision makes a work of Art! [MTC.name]... [MTC.card_desc]</span>") //No period on purpose.
 		qdel(src)
 
 /obj/item/blank_tarot_card/choose //For admins mainly, to spawn a specific tarot card. Not recommended for ruins.

code/game/objects/items/devices/scanners.dm

@@ -384,7 +384,7 @@ SLIME SCANNER
 	var/list/msgs = list()
 	user.visible_message("<span class='warning'>[user] has analyzed the floor's components!</span>", "<span class='warning'>You try to analyze the floor's vitals!</span>")
 	msgs += "<span class='notice'>Analyzing Results for The floor:\n\t Overall Status: Unknown</span>"
-	msgs += "<span class='notice'>\t Damage Specifics: <font color='#FFA500'>[0]</font>/<font color='red>[0]</font></span>"
+	msgs += "<span class='notice'>\t Damage Specifics: <font color='#FFA500'>[0]</font>/<font color='red'>[0]</font></span>"
 	msgs += "<span class='notice'>Key: <font color='#FFA500'>Burns</font><font color ='red'>/Brute</font></span>"
 	msgs += "<span class='notice'>Chassis Temperature: ???</span>"
 	to_chat(user, chat_box_healthscan(msgs.Join("<br>")))

code/game/objects/items/devices/traitordevices.dm

@@ -233,7 +233,7 @@
 	for(var/mob/living/M in fragging_location)//Hit everything in the turf
 		M.apply_damage(20, BRUTE)
 		M.Weaken(6 SECONDS)
-		to_chat(M, "<span_class='warning'>[user] teleports into you, knocking you to the floor with the bluespace wave!</span>")
+		to_chat(M, "<span class='warning'>[user] teleports into you, knocking you to the floor with the bluespace wave!</span>")
 
 /obj/item/paper/teleporter
 	name = "Teleporter Guide"

code/modules/admin/verbs/randomverbs.dm

@@ -951,7 +951,7 @@ Traitors and the like can also be revived with the previous role mostly intact.
 	message_admins("Admin [key_name_admin(usr)] has forced the players to have random appearances.", 1)
 
 	if(notifyplayers == "Yes")
-		to_chat(world, "<span class='notice'><b>Admin [usr.key] has forced the players to have completely random identities!</span>")
+		to_chat(world, "<span class='notice'><b>Admin [usr.key] has forced the players to have completely random identities!</b></span>")
 
 	to_chat(usr, "<i>Remember: you can always disable the randomness by using the verb again, assuming the round hasn't started yet</i>.")
 

code/modules/antagonists/changeling/datum_changeling.dm

@@ -90,7 +90,7 @@ RESTRICT_TYPE(/datum/antagonist/changeling)
 	. += "<span class='danger'>Remember: you get all of their absorbed DNA if you absorb a fellow changeling.</span>"
 
 /datum/antagonist/changeling/farewell()
-	to_chat(owner.current, "<span class='biggerdanger'><B>You grow weak and lose your powers! You are no longer a changeling and are stuck in your current form!</span>")
+	to_chat(owner.current, "<span class='biggerdanger'><b>You grow weak and lose your powers! You are no longer a changeling and are stuck in your current form!</b></span>")
 
 /datum/antagonist/changeling/apply_innate_effects(mob/living/mob_override)
 	var/mob/living/L = ..()

code/modules/antagonists/changeling/powers/mutations.dm

@@ -436,7 +436,7 @@
 	if(remaining_uses < 1)
 		if(ishuman(loc))
 			var/mob/living/carbon/human/H = loc
-			H.visible_message("<span class='warning'>With a sickening crunch, [H] reforms [H.p_their()] shield into an arm!</span>", "<span class='notice'>We assimilate our shield into our body</span>", "<span class='italics>You hear organic matter ripping and tearing!</span>")
+			H.visible_message("<span class='warning'>With a sickening crunch, [H] reforms [H.p_their()] shield into an arm!</span>", "<span class='notice'>We assimilate our shield into our body</span>", "<span class='italics'>You hear organic matter ripping and tearing!</span>")
 			playsound(loc, 'sound/effects/bone_break_2.ogg', 100, TRUE)
 			H.unEquip(src, 1)
 		qdel(src)

code/modules/antagonists/vampire/vampire_powers/hemomancer_powers.dm

@@ -407,7 +407,7 @@
 		owner.AdjustWeakened(-2 SECONDS)
 		owner.AdjustKnockDown(-2 SECONDS)
 		if(drain_amount == 10)
-			to_chat(H, "<span class='warning'>You feel your life force draining!</b></span>")
+			to_chat(H, "<span class='warning'><b>You feel your life force draining!</b></span>")
 
 		if(beam_number >= max_beams)
 			break

code/modules/antagonists/zombie/zombie_spells.dm

@@ -112,7 +112,7 @@
 		to_chat(user, "<span class='warning zombie'>[target]'s brains are blocked.</span>")
 		return // Armor blocks zombies trying to eat your brains!
 
-	to_chat(target, "<span class='userdanger'[user]'s claws dig into your [brain_holder.encased]!</span>")
+	to_chat(target, "<span class='userdanger'>[user]'s claws dig into your [brain_holder.encased]!</span>")
 	user.visible_message("<span class='danger'>[user] digs their claws into [target]'s [brain_holder.name]!</span>", "<span class='danger zombie'>We dig into [target]'s [brain_holder.encased ? brain_holder.encased : brain_holder]...</span>")
 	playsound(user, 'sound/weapons/armblade.ogg', 50, TRUE)
 	if(!do_mob(user, target, 3 SECONDS))

code/modules/atmospherics/machinery/airalarm.dm

@@ -1132,7 +1132,7 @@
 	. = ..()
 	switch(buildstage)
 		if(AIR_ALARM_FRAME)
-			. += "<span class='notice'>Its <i>circuit</i> is missing and the <b>bolts<b> are exposed.</span>"
+			. += "<span class='notice'>Its <i>circuit</i> is missing and the <b>bolts</b> are exposed.</span>"
 		if(AIR_ALARM_UNWIRED)
 			. += "<span class='notice'>The frame is missing <i>wires</i> and the control circuit can be <b>pried out</b>.</span>"
 		if(AIR_ALARM_READY)

code/modules/clothing/under/accessories/holster.dm

@@ -63,7 +63,7 @@
 		to_chat(user, "<span class='warning'>You need an empty hand to draw [holstered]!</span>")
 	else
 		if(user.a_intent == INTENT_HARM)
-			usr.visible_message("<span class='warning'>[user] draws [holstered], ready to shoot!</span></span>", \
+			usr.visible_message("<span class='warning'>[user] draws [holstered], ready to shoot!</span>", \
 			"<span class='warning'>You draw [holstered], ready to shoot!</span>")
 		else
 			user.visible_message("<span class='notice'>[user] draws [holstered], pointing it at the ground.</span>", \

code/modules/food_and_drinks/kitchen_machinery/gibber.dm

@@ -45,7 +45,7 @@
 /obj/machinery/gibber/suicide_act(mob/living/user)
 	if(occupant || locked)
 		return FALSE
-	user.visible_message("<span class='danger'>[user] climbs into [src] and turns it on!</b></span>")
+	user.visible_message("<span class='danger'><b>[user] climbs into [src] and turns it on!</b></span>")
 	user.Stun(20 SECONDS)
 	user.forceMove(src)
 	occupant = user

code/modules/hydroponics/grown/flowers.dm

@@ -168,7 +168,7 @@
 	throw_range = 3
 
 /obj/item/grown/sunflower/attack(mob/M, mob/user)
-	to_chat(M, "<font color='green'><b>[user] smacks you with a sunflower!</font><font color='yellow'><b> FLOWER POWER<b></font>")
+	to_chat(M, "<font color='green'><b>[user] smacks you with a sunflower!</b></font><font color='yellow'><b>FLOWER POWER</b></font>")
 	to_chat(user, "<font color='green'>Your sunflower's </font><font color='yellow'><b>FLOWER POWER</b></font><font color='green'> strikes [M]</font>")
 
 // Moonflower

code/modules/martial_arts/adminfu.dm

@@ -9,7 +9,7 @@
 	if(!D.stat)//do not kill what is dead...
 		A.do_attack_animation(D)
 		D.visible_message("<span class='warning'>[A] manifests a large glowing toolbox and shoves it in [D]'s chest!</span>", \
-							"<spac class='userdanger'>[A] shoves a mystical toolbox in your chest!</span>")
+							"<span class='userdanger'>[A] shoves a mystical toolbox in your chest!</span>")
 		D.death()
 
 		return TRUE

code/modules/martial_arts/cqc.dm

@@ -72,7 +72,7 @@
 		D.apply_damage(10 * damage_multiplier, OXY)
 		D.LoseBreath(3 SECONDS)
 		if(D.getOxyLoss() >= 50 || D.health <= 20)
-			D.visible_message("<span class ='danger>[A] puts [D] to sleep!</span>", \
+			D.visible_message("<span class='danger'>[A] puts [D] to sleep!</span>", \
 						"<span class='userdanger'>[A] knocks you out cold!</span>")
 			D.SetSleeping(40 SECONDS)
 			drop_chokehold()

code/modules/martial_arts/krav_maga.dm

@@ -56,7 +56,7 @@
 		to_chat(owner, "<span class='warning'>You can't use Krav Maga while you're incapacitated.</span>")
 		return
 	if(!owner.get_num_legs())
-		to_chat(owner, "<span class='warning'>You can't leg sweep someone if you have no legs.</spawn>")
+		to_chat(owner, "<span class='warning'>You can't leg sweep someone if you have no legs.</span>")
 		return
 	to_chat(owner, "<b><i>Your next attack will be a Leg Sweep.</i></b>")
 	owner.visible_message("<span class='danger'>[owner] assumes the Leg Sweep stance!</span>")

code/modules/mining/lavaland/loot/tendril_loot.dm

@@ -422,7 +422,7 @@
 			Z.can_destroy = TRUE
 			qdel(Z)
 	else
-		to_chat(user, "<span class'warning'>[src] is still recharging.</span>")
+		to_chat(user, "<span class='warning'>[src] is still recharging.</span>")
 
 /obj/effect/immortality_talisman
 	icon_state = "blank"

code/modules/mob/hear_say.dm

@@ -184,7 +184,7 @@
 				heardword = html_encode(copytext(heardword, 2))
 			if(copytext(heardword,-1) in punctuation)
 				heardword = html_encode(copytext(heardword, 1, length(heardword)))
-			heard = "<span class='game say'>...<i>You hear something about<i>... '[heardword]'...</span>"
+			heard = "<span class='game say'>...<i>You hear something about</i>... '[heardword]'...</span>"
 		else
 			heard = "<span class='game say'>...<i>You almost hear something...</i>...</span>"
 	else

code/modules/mob/living/carbon/carbon_procs.dm

@@ -78,7 +78,7 @@
 
 				for(var/mob/M in viewers(user, null))
 					if(M.client)
-						M.show_message(text("<span class='warning'><B>[user] attacks [src]'s stomach wall with [I]!</span>"), 2)
+						M.show_message(text("<span class='warning'><b>[user] attacks [src]'s stomach wall with [I]!</b></span>"), 2)
 				playsound(user.loc, 'sound/effects/attackblob.ogg', 50, 1)
 
 				if(prob(getBruteLoss() - 50))

code/modules/mob/living/carbon/superheroes.dm

@@ -209,7 +209,7 @@
 	recruiting = FALSE
 	to_chat(user, "<span class='notice'>You have recruited <b>[target]</b> as your henchman!</span>")
 	to_chat(target, "<span class='deadsay'><b>You have decided to enroll as a henchman for [user]. You are now part of the feared 'Greyshirts'.</b></span>")
-	to_chat(target, "<span class='deadsay'><b>You must follow the orders of [user], and help [user.p_them()] succeed in [user.p_their()] dastardly schemes.</span>")
+	to_chat(target, "<span class='deadsay'><b>You must follow the orders of [user], and help [user.p_them()] succeed in [user.p_their()] dastardly schemes.</b></span>")
 	to_chat(target, "<span class='deadsay'>You may not harm other Greyshirt or [user]. However, you do not need to obey other Greyshirts.</span>")
 	SSticker.mode.greyshirts += target.mind
 	target.set_species(/datum/species/human)

code/modules/mob/living/silicon/robot/robot_life.dm

@@ -41,7 +41,7 @@
 /mob/living/silicon/robot/proc/enter_low_power_mode()
 	low_power_mode = TRUE
 	playsound(src, "sound/mecha/lowpower.ogg", 50, FALSE, SOUND_RANGE_SET(10))
-	to_chat(src, "<span_class='warning'>Alert: Power cell requires immediate charging.</span>")
+	to_chat(src, "<span class='warning'>Alert: Power cell requires immediate charging.</span>")
 	handle_no_power()
 
 /mob/living/silicon/robot/proc/handle_equipment()

code/modules/mob/living/simple_animal/hostile/mining/elites/elite.dm

@@ -407,7 +407,7 @@ While using this makes the system rely on OnFire, it still gives options for tim
 		text += "<b>Bear in mind, if anyone interacts with your tumor, you'll be resummoned here to carry out another fight. In such a case, you will regain your full max health.</b>"
 		text += "<b>Also, be wary of your fellow inhabitants, they likely won't be happy to see you! \n</b>"
 		text += "<span class='big bold'>Note that you are a lavaland monster, and thus not allied to the station.</span>"
-		text += "<span class='big bold>'You should not cooperate or act friendly with any station crew unless under extreme circumstances!</span>"
+		text += "<span class='big bold'>You should not cooperate or act friendly with any station crew unless under extreme circumstances!</span>"
 		text += "<span class='warning'>Do not attack the Mining Station or Labour Camp, unless the Shaft Miner you are actively fighting runs into the Station/Camp.</span>"
 		text += "<span class='warning'>After they are killed, you must withdraw. If you wish to continue attacking the Station, you MUST ahelp.</span>"
 		text += "<span class='warning'>If teleported to the Station by jaunter, you are allowed to attack people on Station, until you get killed.</span>"

code/modules/newscaster/obj/newspaper.dm

@@ -99,7 +99,7 @@
 				for(var/datum/feed_channel/NP in news_content)
 					pages++
 				if(important_message!=null)
-					dat += "<div style='float:center;'><font size=4><b>Wanted Issue:</b></font size></div><br><br>"
+					dat += "<div style='float:center;'><font size=4><b>Wanted Issue:</b></font></div><br><br>"
 					dat += "<b>Criminal name</b>: <font color='maroon'>[important_message.author]</font><br>"
 					dat += "<b>Description</b>: [important_message.body]<br>"
 					dat += "<b>Photo:</b>: "

code/modules/paperwork/fax.dm

@@ -36,7 +36,7 @@ GLOBAL_LIST_EMPTY(adminfaxes)
 	html += "<div class='block'>"
 	html += "<h2>Admin Faxes</h2>"
 	html += "<table>"
-	html += "<tr style='font-weight:bold;'><td width='150px'>Name</td><td width='150px'>From Department</td><td width='150px'>To Department</td><td width='75px'>Sent At</td><td width='150px'>Sent By</td><td width='50px'>View</td><td width='50px'>Reply</td><td width='75px'>Replied To</td></td></tr>"
+	html += "<tr style='font-weight:bold;'><td width='150px'>Name</td><td width='150px'>From Department</td><td width='150px'>To Department</td><td width='75px'>Sent At</td><td width='150px'>Sent By</td><td width='50px'>View</td><td width='50px'>Reply</td><td width='75px'>Replied To</td></tr>"
 	for(var/datum/fax/admin/A in GLOB.adminfaxes)
 		html += "<tr>"
 		html += "<td>[A.name]</td>"
@@ -65,7 +65,7 @@ GLOBAL_LIST_EMPTY(adminfaxes)
 	html += "<div class='block'>"
 	html += "<h2>Departmental Faxes</h2>"
 	html += "<table>"
-	html += "<tr style='font-weight:bold;'><td width='150px'>Name</td><td width='150px'>From Department</td><td width='150px'>To Department</td><td width='75px'>Sent At</td><td width='150px'>Sent By</td><td width='175px'>View</td></td></tr>"
+	html += "<tr style='font-weight:bold;'><td width='150px'>Name</td><td width='150px'>From Department</td><td width='150px'>To Department</td><td width='75px'>Sent At</td><td width='150px'>Sent By</td><td width='175px'>View</td></tr>"
 	for(var/datum/fax/F in GLOB.faxes)
 		html += "<tr>"
 		html += "<td>[F.name]</td>"