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

Metal mask not mask. #8

Open
MrBobCool opened this issue Jun 20, 2018 · 4 comments
Open

Metal mask not mask. #8

MrBobCool opened this issue Jun 20, 2018 · 4 comments

Comments

@MrBobCool
Copy link

Made a pull request on Gitbucket or whatever it was called about this. Metal mask go into the head slot not the mask slot. Clearly says mask and it even states "Its a metal hockey mask" and shit. Please fix daddy. I can't let it ruin my immmerrssioooon!

@Digdugxx
Copy link
Collaborator

On it

@LarissaMayrink
Copy link
Collaborator

The metal mask has armor values, it shouldn't go into he mask slot, and, it is clearly also a helmet as it covers the entire head. Same goes for the welding mask which also goes on the head slot.

@Digdugxx if you already changed it just revert

@MrBobCool
Copy link
Author

Mhm. The welding helmet can go up and down. Its a metal mask keywords mask. If it is a mask it should be able to move up and down like a welding mask.

@LarissaMayrink
Copy link
Collaborator

too much of a unecessary mess to code if we want the armor values to be nonexistant while the mask is up, that plus the mask has no reason to be up except to make it confusing for people if the armor values goes away when its up, also it is based on the raider arclight helmet, which can't even physically move it, while the non raider alternative to it is the f3 welder's mask which still has no movement, it might be better to just slightly change the sprites.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants