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

Construction recipe for barred window with curtains #36797

Merged
merged 5 commits into from
Jan 8, 2020

Conversation

Kilvoctu
Copy link
Contributor

@Kilvoctu Kilvoctu commented Jan 7, 2020

Summary

SUMMARY: Content "Construction recipe for barred window with curtains"

Purpose of change

I can build a window, a window with curtains, and add bars onto the window. I can't add bars and a curtain? Don't see why it shouldn't be possible.

Describe the solution

Extend the current stages of "Install Bars Onto Window" so curtains can be affixed at the end.

Describe alternatives you've considered

Do nothing and continue to have elaborate setups of barred windows and door curtains.

Testing

Built a window, installed bars onto the window, then installed curtains onto the barred window. Peeked through the curtains, opened and closed the curtains to test line of sight. Tried walking through the window when it was open and closed. Tore down the curtains.

Additional context

Recipe is the same as installing curtains onto a normal window.
Smashing this construction will revert it back to a normal curtainless window with bars.
Graphically will look like barred window when open and curtains when closed.

Add conditional handler for tearing down curtains from a barred window
No shortcuts... download a compiler, learn c++, and finish what I started.
Game compiles, load save, no error, tear down curtain and they stayed down
@John-Candlebury John-Candlebury added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. labels Jan 8, 2020
@I-am-Erk I-am-Erk added the [C++] Changes (can be) made in C++. Previously named `Code` label Jan 8, 2020
{
"type": "terrain",
"id": "t_window_bars_curtains",
"name": "window with metal bars and curtains.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"name": "window with metal bars and curtains.",
"name": "window with metal bars and curtains",

{
"type": "terrain",
"id": "t_window_bars_domestic",
"name": "window with metal bars and curtains.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"name": "window with metal bars and curtains.",
"name": "window with metal bars and curtains",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Commited these changes and astyle test

learn what astyle is and attempt application. minor fix, remove unneeded "."'s
@I-am-Erk I-am-Erk merged commit 0e87154 into CleverRaven:master Jan 8, 2020
@Kilvoctu Kilvoctu deleted the changes branch January 15, 2020 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants