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

Fix screen's overcovering of areas (fixes #14) #16

Closed
wants to merge 1 commit into from
Closed

Fix screen's overcovering of areas (fixes #14) #16

wants to merge 1 commit into from

Conversation

kilbith
Copy link

@kilbith kilbith commented Mar 29, 2016

This pull-request fixes the overcovering of areas on the HUD (#14), making the game uncomfortable when the owners are numerous.

  • Sub-owners are concealed and counted below the first entry of AreaStrings.
  • A new command (/list_owners) is added to display all owners of an area where the player stands inside.
  • Miscellaneous clean-ups and misspelling fixes.

screenshot_20160329_155602

@kilbith
Copy link
Author

kilbith commented Apr 4, 2016

@ShadowNinja Do you acknowledge receipt of this PR ? My fork might get deleted if nothing happens in the following months.

@ShadowNinja
Copy link
Member

@kilbith: Yep. Note that your changes only handle one parent when there and be multiple areas surrounding a point of which none are sub-areas of the others.

@kilbith
Copy link
Author

kilbith commented Apr 14, 2016

@ShadowNinja Sorry, I deleted my fork due to lack of motivation. If you intend to bring some modifications to these features, do them on your own.

@SmallJoker
Copy link
Member

Dead PR.

@SmallJoker SmallJoker closed this Feb 2, 2020
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

Successfully merging this pull request may close these issues.

3 participants