-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes yet more runtimes. Makes click catcher screen objs shared. #23428
Conversation
code/modules/client/preferences.dm
Outdated
@@ -50,6 +50,10 @@ datum/preferences | |||
load_preferences() | |||
load_and_update_character() | |||
sanitize_preferences() | |||
world.log << client.mob ? client.mob.type : "No mob" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left-over debug log?
var/output = "<div align='center'>" | ||
/mob/new_player/proc/new_player_panel_proc(force = FALSE) | ||
if(!SScharacter_setup.initialized && !force) | ||
return // Not ready yet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the player told in any way that the panel can't open?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will automatically be opened at ss init; the player doesn't need to see anything here, this just means that client/New()
ran before SScharacter_setup/Initialize()
.
|
||
/mob/new_player/proc/new_player_panel_proc() | ||
var/output = "<div align='center'>" | ||
/mob/new_player/proc/new_player_panel_proc(force = FALSE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why _proc
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Legacy name, due to some old verb/proc hack (unclear the intention). Probably should rename.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, didn't actually see the old one.
7442b9d
to
c646255
Compare
…en clients. Fixes error with prefs and the new player window.
c646255
to
7fafff3
Compare
Fixes #23102.
Fixes #21311 (the muzzle there is obsolete, currently occurs on live in holograms. will likely be reopned with other culprits).
Closes #20414 (not clear if this will fix it).
Fixes #23423.
Note that currently, click catchers are added to the client inconsistently due to above runtime.
This will make the new player panel (with buttons like Ready, Character Setup, etc.) not come up until the character setup SS inits (it's currently not interactive until approximately then anyway).