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

Trenching. #134

Closed
kevingranade opened this issue Mar 1, 2013 · 4 comments

Comments

Projects
None yet
3 participants
@kevingranade
Copy link
Member

commented Mar 1, 2013

Currently adjacent pits are separate, we'd like them to instead act as trenches, partially obscuring occupants and items from those outside and vice versa, and allowing free movement between trench sections, and easy movement from pit <-> shallow pit <-> ground.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@sparr

This comment has been minimized.

Copy link
Member

commented Mar 31, 2015

Would it be appropriate to reuse or generalize the code currently used for combining graphic tiles for adjacent terrain here?

@kevingranade

This comment has been minimized.

Copy link
Member Author

commented Apr 1, 2015

@sparr

This comment has been minimized.

Copy link
Member

commented Apr 1, 2015

I mean, not just using it to draw them, but also to decide if/when they are combined for gameplay purposes, too, and how moving between them should be allowed.

@illi-kun

This comment has been minimized.

Copy link
Member

commented Nov 20, 2016

This issue was closed as it appears inactive.

Reducing open issues to those which are (or will) be actively worked upon helps us focus our efforts. This issue has not been deleted - it still appears in searches and if it contains relevant information you are encouraged to continue to link to it.

If this issue was a bug

It should be reopened if it can be reproduced in the current build. You can obtain the most recent copy here. Please check there is not a more recent report of this bug before doing so. If no more recent report exists you should continue the discussion in this issue.

If this was a feature request

If the consensus was that the idea was good you could consider submitting an implementation via a PR. If you want to comment further please do so here as opposed to opening a new issue. Before posting check nobody has already made the same point and consider whether your comments are likely to lead to an implementation. If you have doubts about either consider instead voting for the issue

If you want to work on this issue

Then either assign it to yourself or if you are unable to do so claim it via adding a comment. Please don't assign others or make a general request for action.

@illi-kun illi-kun closed this Nov 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.