Skip to content

[Vanilla Fix] Correct sidebar radar horizontal offset for both styles - #2458

Merged
DeathFishAtEase merged 11 commits into
Phobos-developers:developfrom
Super-StarX:fix-sidebar-radar-offset
Oct 9, 2026
Merged

DeathFishAtEase merged 11 commits into
Phobos-developers:developfrom
Super-StarX:fix-sidebar-radar-offset

Conversation

@Super-StarX

@Super-StarX Super-StarX commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

What kind of change is this?

  • New feature, vanilla bugfix or enhancement of a released feature - changelog, docs and credits entries are needed.
  • Improvement to a new (unreleased) feature - docs and credits entries are needed; no changelog entry (Skip Changelog).
  • Bugfix to a new (unreleased) feature - credits entry is needed; no changelog or docs entries (Skip Changelog, Skip Docs).
  • Bugfix to an old (released) feature - changelog and credits entries are needed; no docs entry (Skip Docs).
  • Completely minor change (e.g. a typo fix) - no entries are needed (Skip Changelog, Skip Docs, Skip Credits).

Description

Vanilla initializes the sidebar radar X origin at 16. The visible aperture starts at 13 for the Allied/GDI sidebar and 14 for the Soviet/NOD sidebar, so the radar content, its input rectangle, and the view rectangle are offset by 3 and 2 pixels respectively.

This PR corrects RadarClass::RadarX after the Allied and Soviet sidebar branches converge, using the existing isNODSidebar state derived from Sidebar.GDIPositions. The fix is enabled by default and can be disabled with:

[AudioVisual]
UseSidebarRadarOffsetFix=no

The before/after comparison was captured on the map Super Big Trouble (超级大麻烦).

Before

Before

After

After

Notes

@Super-StarX
Super-StarX force-pushed the fix-sidebar-radar-offset branch 4 times, most recently from 262a6d9 to 7eeace5 Compare October 7, 2026 13:06
@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

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.

@Super-StarX
Super-StarX force-pushed the fix-sidebar-radar-offset branch from 7eeace5 to 9864982 Compare October 7, 2026 13:23

@NetsuNegi NetsuNegi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@NetsuNegi NetsuNegi added ❓Vanilla bug Vanilla game bugs that are requested to be fixed Needs testing ⚙️T1 T1 maintainer review is sufficient labels Oct 7, 2026
@DeathFishAtEase

Copy link
Copy Markdown
Collaborator

Before:
Before

After:
After

@DeathFishAtEase DeathFishAtEase added Tested Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. and removed Needs testing labels Oct 8, 2026
@Super-StarX Super-StarX changed the title [Vanilla Fix] Correct sidebar radar horizontal offset [Vanilla Fix] Correct sidebar radar horizontal offset for both styles Oct 8, 2026
Comment thread docs/Whats-New.md Outdated
Comment thread src/Ext/Side/Hooks.SidebarGDIPositions.cpp
@DeathFishAtEase DeathFishAtEase added On Hold Due to some reasons, it cannot be completed temporarily and removed Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. labels Oct 8, 2026
@DeathFishAtEase DeathFishAtEase added Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. and removed On Hold Due to some reasons, it cannot be completed temporarily labels Oct 8, 2026
@DeathFishAtEase

DeathFishAtEase commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

[Test] (In the latest Nightly Build)

Default behavior

  • By default, the map frames of both ALL and SOV are aligned to the radar map centers of their corresponding sides (correspondingly, the map frames of the two sides differ by 1 pixel).
Lastest

Sidebar.RadarOffsetFix

  • For mods that have previously fixed the misalignment of radar map centers, those sides with Sidebar.GDIPositions=false can also manually set an offset value via Sidebar.RadarOffsetFix, thereby fixing the map frame out-of-bounds issue while maintaining consistency with the map frame position of the ALL side.
Sidebar RadarOffsetFix

@DeathFishAtEase
DeathFishAtEase merged commit e8ec07f into Phobos-developers:develop Oct 9, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️T1 T1 maintainer review is sufficient Tested ❓Vanilla bug Vanilla game bugs that are requested to be fixed Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants