Merge pull request #11074 from GinjaNinja32/title

Title screen update

code/modules/mob/new_player/login.dm

@@ -1,3 +1,9 @@
+
+/var/atom/movable/lobby_image = new /atom/movable{icon = 'icons/misc/title.dmi'; icon_state = "title"; screen_loc = "1,1"; name = "Baystation12"}
+
+/mob/new_player
+	var/client/my_client // Need to keep track of this ourselves, since by the time Logout() is called the client has already been nulled
+
 /mob/new_player/Login()
 	update_Login_details()	//handles setting lastKnownIP and computer_id for use by the ban systems as well as checking for multikeying
 	if(join_motd)
@@ -8,24 +14,12 @@
 		mind.active = 1
 		mind.current = src
 
-	if(length(newplayer_start))
-		loc = pick(newplayer_start)
-	else
-		loc = locate(1,1,1)
-	lastarea = loc
-
+	loc = null
+	client.screen += lobby_image
+	my_client = client
 	sight |= SEE_TURFS
 	player_list |= src
 
-/*
-	var/list/watch_locations = list()
-	for(var/obj/effect/landmark/landmark in landmarks_list)
-		if(landmark.tag == "landmark*new_player")
-			watch_locations += landmark.loc
-
-	if(watch_locations.len>0)
-		loc = pick(watch_locations)
-*/
 	new_player_panel()
 	spawn(40)
 		if(client)

code/modules/mob/new_player/logout.dm

@@ -1,7 +1,13 @@
 /mob/new_player/Logout()
 	ready = 0
+
+	// see login.dm
+	if(my_client)
+		my_client.screen -= lobby_image
+		my_client = null
+
 	..()
 	if(!spawning)//Here so that if they are spawning and log out, the other procs can play out and they will have a mob to come back to.
 		key = null//We null their key before deleting the mob, so they are properly kicked out.
 		qdel(src)
-	return
+	return