Second pass at representing stacked information

Lots of internal discussion on my last approach, so here's a cleaner alternative
- No more offset with two complex flickering graphics on top of each other
- Instead, "less common" secondary info is represented by a coloured dot overlaid on the primary icon
    - Red for SPB Attack, Turquoise for Mystic Melody
- Only show Spray Can/Mystic Melody for Race-compatible courses
- Show courses where you haven't picked up a Spray Can with a dot

src/k_menudraw.c

@@ -3294,11 +3294,10 @@ void M_DrawCupSelect(void)
 		struct work_array_t {
 			emblem_t *medal;
 			UINT16 col;
-			UINT8 mapvisited;
+			UINT16 dotcol;
 		} work_array[CUPCACHE_MAX];
 
 		boolean incj = false;
-		boolean showalternate = (skullAnimCounter & 4);
 
 		i = j = 0;
 
@@ -3313,8 +3312,7 @@ void M_DrawCupSelect(void)
 				incj = false;
 
 				work_array[j].medal = NULL;
-				work_array[j].col = UINT16_MAX;
+				work_array[j].col = work_array[j].dotcol = UINT16_MAX;
-				work_array[j].mapvisited = 0;
 
 				if (templevelsearch.timeattack)
 				{
@@ -3344,29 +3342,27 @@ void M_DrawCupSelect(void)
 						{
 							incj = true;
 
-							if ((gamedata->collected[emblem-emblemlocations])
+							if (gamedata->collected[emblem-emblemlocations])
-							&& showalternate)
 							{
-								work_array[j].medal = emblem;
+								work_array[j].dotcol = M_GetEmblemColor(emblem);
 							}
 						}
 
 						emblem = M_GetLevelEmblems(-1);
 					}
 				}
-				else
+				else if ((gamedata->gotspraycans > 0) && (mapheaderinfo[map]->typeoflevel & TOL_RACE))
 				{
+					incj = true;
+
 					if (mapheaderinfo[map]->cache_spraycan < gamedata->numspraycans)
 					{
 						work_array[j].col = gamedata->spraycans[mapheaderinfo[map]->cache_spraycan].col;
-						incj = true;
 					}
 
-					if ((mapheaderinfo[map]->records.mapvisited & MV_MYSTICMELODY)
+					if (mapheaderinfo[map]->records.mapvisited & MV_MYSTICMELODY)
-						&& (incj == false || showalternate))
 					{
-						work_array[j].mapvisited |= MV_MYSTICMELODY;
+						work_array[j].dotcol = SKINCOLOR_TURQUOISE;
-						incj = true;
 					}
 				}
 
@@ -3391,24 +3387,46 @@ void M_DrawCupSelect(void)
 			y--;
 
 			for (i = 0; i < j; i++)
+			{
+				if (templevelsearch.timeattack)
 				{
 					if (work_array[i].medal)
 					{
+						// Primary Medal
+
 						V_DrawMappedPatch(x, y, 0, W_CachePatchName(M_GetEmblemPatch(work_array[i].medal, false), PU_CACHE),
 							R_GetTranslationColormap(TC_DEFAULT, M_GetEmblemColor(work_array[i].medal), GTC_MENUCACHE));
 					}
-				else if (work_array[i].mapvisited & MV_MYSTICMELODY)
+					else
 					{
-					V_DrawScaledPatch(x, y, 0, W_CachePatchName("GOTMEL", PU_CACHE));
+						// Need it!
+
+						V_DrawScaledPatch(x, y, 0, W_CachePatchName("NEEDIT", PU_CACHE));
 					}
-				else if (work_array[i].col < numskincolors)
+				}
+				else
 				{
+					if (work_array[i].col < numskincolors)
+					{
+						// Spray Can
+
 						V_DrawMappedPatch(x, y, 0, W_CachePatchName("GOTCAN", PU_CACHE),
 							R_GetTranslationColormap(TC_DEFAULT, work_array[i].col, GTC_MENUCACHE));
 					}
 					else
 					{
-					V_DrawScaledPatch(x, y, 0, W_CachePatchName("NEEDIT", PU_CACHE));
+						// Need it!
+
+						V_DrawFill(x+3, y+3, 2, 2, 20);
+					}
+				}
+
+				if (work_array[i].dotcol < numskincolors)
+				{
+					// Bonus (Secondary Medal or Ancient Shrine)
+
+					V_DrawMappedPatch(x+4, y+7, 0, W_CachePatchName("COLORSP1", PU_CACHE),
+						R_GetTranslationColormap(TC_DEFAULT, work_array[i].dotcol, GTC_MENUCACHE));
 				}
 
 				x += 10;