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

Overhaul sights' volumes #58588

Merged
merged 2 commits into from Jun 28, 2022
Merged

Conversation

l29ah
Copy link
Contributor

@l29ah l29ah commented Jun 20, 2022

Summary

Content "Overhaul sights' volumes"

Purpose of change

Making them more realistic. Also fixes #53788

Describe the solution

Googling all the generic sights, preferably technical drawings, and eyeballing their volume.

Describe alternatives you've considered

Testing

Additional context

Making them more realistic
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jun 20, 2022
@SirPendrak
Copy link
Contributor

For detailed holographic sight size see #53788

@bombasticSlacks
Copy link
Contributor

for a few of these it might be worth it to include a comment with a link / name for the model you used. So someone else doesn't come in, in 6 months and "Overhaul sights' volumes" again 😆

@ZhilkinSerg ZhilkinSerg merged commit cffcccd into CleverRaven:master Jun 28, 2022
Drew4484 pushed a commit to Drew4484/Cataclysm-DDA that referenced this pull request Jul 4, 2022
Co-authored-by: Maleclypse <54345792+Maleclypse@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Holographic sight is too big
5 participants