Skip to content
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

Inconsistencies with toggling lights while inside other things #1083

Closed
3 tasks done
DoomLaser9 opened this issue Sep 18, 2022 · 3 comments · Fixed by #3248
Closed
3 tasks done

Inconsistencies with toggling lights while inside other things #1083

DoomLaser9 opened this issue Sep 18, 2022 · 3 comments · Fixed by #3248
Labels
Bug Admin events aren't bugs Consistency Issue Who needs consistency?

Comments

@DoomLaser9
Copy link
Contributor

Description

Messages do not take into account current light status, have inconsistent formatting, and do not apply to all lights. Rail lights also fail to create light when turned on from inside things.

Difference between expected and actual behavior

You get the message You cannot turn the light on while in the [...] (whatever you're in is on the end of the message). However, you get the same message if the light is already on; in that case, it should tell you that you cannot turn the light off. The affected code is in flashlight.dm, hardhat.dm, misc_special.dm, and marine_armor.dm.

Note that the message only has SPAN_WARNING for armor, and not for anything else (inconsistency).

Also, rail lights on guns can still freely be toggled while inside things and has inconsistent behavior with lighting. For example, going inside something with the rail light on usually maintains light around you, but if you switch it off and back on, the light around you doesn't come back on.

Steps to reproduce

  • Turn on flashlight, get in closet, toggle flashlight, notice message saying you can't turn it on.
  • Turn on rail light on gun, get in closet, turn rail light off and back on, notice that light around closet is now gone (i.e. you're now in darkness)

Issue bingo (not required to be all checked)

@usnpeepoo usnpeepoo added Bug Admin events aren't bugs Consistency Issue Who needs consistency? labels Oct 12, 2022
@Huffie56
Copy link
Contributor

Huffie56 commented May 6, 2023

bug still active.
image

@Huffie56
Copy link
Contributor

Huffie56 commented May 6, 2023

Will work on fixing that.

@Huffie56
Copy link
Contributor

Huffie56 commented May 6, 2023

@ItsVyzo can you link this PR #3248 to this issue?
thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs Consistency Issue Who needs consistency?
Projects
None yet
3 participants