From 1f6ec81e4039e35e298b642624a23e5fda8efcd0 Mon Sep 17 00:00:00 2001 From: Ccomp5950 Date: Thu, 8 May 2014 00:52:04 -0500 Subject: [PATCH] Jobs you can't select are struckthrough instead of just being red. Before: It's going to be kinda confusing when a new player goes to setup their jobs and they see this: http://i.imgur.com/miK9Isk.png Specifically "unavailable occupations are in red" and then a list of days till they can play that role but also RED [NEVER] which to a new player migth cause confusion thinking they can never play that role. After: Remove the red and add a strikethrough ( http://i.imgur.com/BQ90xr6.png ) to show "these jobs are unavailable" while making the other jobs look, well more available. --- code/modules/client/preferences.dm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/code/modules/client/preferences.dm b/code/modules/client/preferences.dm index 6a4a81e141..e85d70705f 100644 --- a/code/modules/client/preferences.dm +++ b/code/modules/client/preferences.dm @@ -418,7 +418,7 @@ datum/preferences var/HTML = "" HTML += "
" - HTML += "Choose occupation chances
Unavailable occupations are in red.

" + HTML += "Choose occupation chances
Unavailable occupations are crossed out.

" HTML += "
\[Done\]

" // Easier to press up here. HTML += "
" // Table within a table for alignment, also allows you to easily add more colomns. HTML += "" @@ -443,11 +443,11 @@ datum/preferences var/rank = job.title lastJob = job if(jobban_isbanned(user, rank)) - HTML += "[rank]" + HTML += "[rank]" continue if(!job.player_old_enough(user.client)) var/available_in_days = job.available_in_days(user.client) - HTML += "[rank]" + HTML += "[rank]" continue if((job_civilian_low & ASSISTANT) && (rank != "Assistant")) HTML += "[rank]"
\[BANNED]
\[BANNED]
\[IN [(available_in_days)] DAYS]
\[IN [(available_in_days)] DAYS]