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

Add option to raise the visor to motorcycle helmet #48560

Merged

Conversation

Salty-Panda
Copy link
Contributor

Summary

Content "Add option to raise the visor to motorcycle helmet"

Purpose of change

You should be able to raise the visor

Describe the solution

The new item motorcycle_helmet_raised copies entire entry, replace using copy-from once #43144 will get merged

Describe alternatives you've considered

none

Testing

Tested values in-game

@actual-nh actual-nh added 0.F String Freeze <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON labels Apr 18, 2021
data/json/items/armor/helmets.json Outdated Show resolved Hide resolved
data/json/items/armor/helmets.json Show resolved Hide resolved
data/json/items/armor/helmets.json Show resolved Hide resolved
data/json/items/armor/helmets.json Show resolved Hide resolved
@Night-Pryanik Night-Pryanik deleted the branch CleverRaven:master July 15, 2021 02:44
@Maleclypse
Copy link
Member

Please open a new PR for this targeting the experimental branch. Was closed as a side effect of deleting the 0.F -dev branch

@Night-Pryanik
Copy link
Contributor

Hmm, that's really weird, I didn't delete anything. I don't know why it's saying that was me who deleted the branch.

@kevingranade kevingranade reopened this Jul 15, 2021
@kevingranade kevingranade changed the base branch from 0.F-dev to master July 15, 2021 05:00
@Salty-Panda Salty-Panda force-pushed the add_visor_to_motorcycle_helmet branch from 996a928 to 23d610f Compare August 14, 2021 09:02
@Salty-Panda
Copy link
Contributor Author

Rebased

"material_thickness": 6,
"armor": [
{ "covers": [ "head" ], "coverage": 100, "encumbrance": 25 },
{ "covers": [ "eyes" ], "coverage": 100, "encumbrance": 5 },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if it's possible to set an material thickness specifically for one section, I'd think this would have 1 or 2 thickness when visor down over the eyes. But again not sure that's how it works.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You cannot.

@ZhilkinSerg ZhilkinSerg merged commit cb7433f into CleverRaven:master Aug 15, 2021
@Salty-Panda Salty-Panda deleted the add_visor_to_motorcycle_helmet branch August 15, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants