Skip to content

Conversation

@ywgATustcbbs
Copy link
Contributor

This pr only blocks extra view. Egg is not an alien type. No idea if it should be fixed. Therefore, leave it as is.
Needs another pr to block jump jets, blink, nade etc.

Tested on my client. Works fine.

@ywgATustcbbs ywgATustcbbs changed the title Do not draw aliens beyond 16:9 area of the screen. Do not draw aliens outside 16:9 area of the screen. Sep 10, 2025
@mithrand0 mithrand0 self-requested a review September 10, 2025 10:53
@ywgATustcbbs
Copy link
Contributor Author

ywgATustcbbs commented Sep 12, 2025

THis is kinda prototype pr. Due to the current state of the discussions, No control logic is added.

Control logic could be added easily after it comes to a final agreement.

In case the returned value of ShouldRenderableBeIgnored() Is modified by CE or something, it should be protected.

@mithrand0
Copy link
Contributor

mithrand0 commented Sep 13, 2025

Since the proposed changes aim to limit cheating, they will also require substantial additional work in one of our closed-source repositories. For that reason, this PR (and all black-bar, visibility or aim restrictions PR's in general) should not be merged until both components are implemented together.

I propose to convert this PR to a draft, to prevent an accidental merge.

@ywgATustcbbs ywgATustcbbs marked this pull request as draft September 14, 2025 13:10
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.

2 participants