Add drag-to-reposition to territorial domination scoreboard#8040
Open
steeldesade wants to merge 1 commit into
Open
Conversation
Implements draggable repositioning for the TD scoreboard UI via RmlUi mousedown/mousemove/mouseup events on the header area. Position is saved to config and restored on load. Adds cursor: move to header.
Contributor
Integration Test Results15 tests ±0 6 ✅ ±0 3s ⏱️ ±0s For more details on these failures, see this check. Results for commit 0434ccd. ± Comparison against base commit e7d11fd. |
Collaborator
|
needs some time in the oven. Dragging around is finnicky. If the cursor too far it doesn't follow. It's also possible to place the UI outside of the screen space potentially making it really difficult to put back into frame. |
SethDGamre
reviewed
Jun 22, 2026
| local spSetConfigInt = Spring.SetConfigInt | ||
| local spGetConfigInt = Spring.GetConfigInt | ||
|
|
||
| -- Drag state (position saved/loaded via RmlUi <handle> — no Spring mouse callbacks needed) |
Collaborator
There was a problem hiding this comment.
cleanup vibe code comments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem: Territorial Domination UI is locked in place, covering player view and player camera buttons and can not be moved.
Fix: Implements draggable repositioning for the TD scoreboard UI via RmlUi mousedown/mousemove/mouseup events on the header area. Position is saved to config and restored on load. Adds cursor: move to header.
AI / LLM usage statement:
Visual Studio and Claude Code