Skip to content

[Minor] Add global default values for some tags#2165

Open
Coronia wants to merge 9 commits intoPhobos-developers:developfrom
Coronia:global-tag
Open

[Minor] Add global default values for some tags#2165
Coronia wants to merge 9 commits intoPhobos-developers:developfrom
Coronia:global-tag

Conversation

@Coronia
Copy link
Copy Markdown
Contributor

@Coronia Coronia commented Apr 7, 2026

If there's other tag that is worth globalizing please let me know

@Coronia Coronia added Minor Minor feature and/or fix, not a lot of changes or they are not significant ⚙️T1 T1 maintainer review is sufficient No Documentation Needed No documentation needed whatsoever No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. labels Apr 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026 •
edited
Loading

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.

@TaranDahl
Copy link
Copy Markdown
Contributor

TaranDahl commented Apr 7, 2026

Arcing.AllowElevationInaccuracy.
CanBeBuiltOn and IsPassable for TerrainTypes. This needs a smart default, smart to true for non-Tiber tree.
Splits.UseWeaponTargeting, AirburstWeapon.ApplyFirepowerMult
Shrapnel.AffectsGround, Shrapnel.AffectsBuildings, Shrapnel.UseWeaponTargeting

TaranDahl
TaranDahl approved these changes Apr 9, 2026
View reviewed changes
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.

It would be better to wait for a longer period of time and take into account more users' opinions.

DeathFishAtEase

This comment was marked as duplicate.

Sign in to view

DeathFishAtEase
DeathFishAtEase reviewed Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@DeathFishAtEase DeathFishAtEase left a comment

Choose a reason for hiding this comment

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

Repetitive content

Comment thread docs/New-or-Enhanced-Logics.md Outdated
@phoboscn-bot
Copy link
Copy Markdown

phoboscn-bot commented Apr 13, 2026

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/355/1

@TaranDahl
Copy link
Copy Markdown
Contributor

TaranDahl commented Apr 21, 2026

OpenTopped.IgnoreRangefinding
OpenTopped.AllowFiringIfDeactivated
OpenTopped.ShareTransportTarget

@DeathFishAtEase
Copy link
Copy Markdown
Collaborator

DeathFishAtEase commented Apr 21, 2026

OpenTopped.IgnoreRangefinding  OpenTopped.AllowFiringIfDeactivated  OpenTopped.ShareTransportTarget

They seem to have been processed in 0d7ac01? See src/Ext/Rules/Body.cpp.

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

Reviewers

@DeathFishAtEase DeathFishAtEase DeathFishAtEase left review comments

@TaranDahl TaranDahl TaranDahl approved these changes

Assignees

No one assigned

Labels

Minor Minor feature and/or fix, not a lot of changes or they are not significant No Documentation Needed No documentation needed whatsoever No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. ⚙️T1 T1 maintainer review is sufficient

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Coronia @TaranDahl @phoboscn-bot @DeathFishAtEase