Skip to content

[Vanilla Enhancement] Customize reveal radius of RevealToAll#2033

Open
NetsuNegi wants to merge 10 commits intoPhobos-developers:developfrom
NetsuNegi:branch/reveal_to_all_radius
Open

[Vanilla Enhancement] Customize reveal radius of RevealToAll#2033
NetsuNegi wants to merge 10 commits intoPhobos-developers:developfrom
NetsuNegi:branch/reveal_to_all_radius

Conversation

@NetsuNegi
Copy link
Copy Markdown
Contributor

@NetsuNegi NetsuNegi commented Jan 8, 2026

  • In vanilla, RevealToAll is hardcoded to reveal area in radius is Sight. Now you can customize it.

In rulesmd.ini:

[SOMEBUILDING]                  ; BuildingType
RevealToAll.Radius=             ; integer

@NetsuNegi NetsuNegi added Needs testing ⚙️T1 T1 maintainer review is sufficient ❓Unhardcoding / Customization Make something more tweakable labels Jan 8, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 8, 2026

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@NetsuNegi NetsuNegi marked this pull request as draft January 8, 2026 13:02
@NetsuNegi NetsuNegi marked this pull request as ready for review January 8, 2026 13:42
Copy link
Copy Markdown
Contributor

@TaranDahl TaranDahl left a comment

Choose a reason for hiding this comment

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

Thinking that you should also deal with 4ADE68.
There are also some calls to vt_entry_48C related to RevealToAll, in TechnoClass::Limbo and BuildingClass::Destroy. It's uncertain whether they should also be handled.

@Rinimabi
Copy link
Copy Markdown

wrok

[GACSPH]
RevealToAll.Radius=255

2222

@NetsuNegi
Copy link
Copy Markdown
Contributor Author

Thinking that you should also deal with 4ADE68. There are also some calls to vt_entry_48C related to RevealToAll, in TechnoClass::Limbo and BuildingClass::Destroy. It's uncertain whether they should also be handled.

I don't know if that's nessessary

@phoboscn-bot
Copy link
Copy Markdown

To Chinese users:
This pull request has been mentioned on Phobos CN. There might be relevant details there:

致中文用户:
此拉取请求已在 Phobos CN 上被提及。那里可能有相关详细信息:

https://www.phoboscn.top/t/topic/67/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs testing ⚙️T1 T1 maintainer review is sufficient ❓Unhardcoding / Customization Make something more tweakable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants