Skip to content

feat: Added ability to lock a tactical map object to screen - #3514

Merged
alexanderson1993 merged 1 commit into
developfrom
feat/add-stay-on-screen
Jun 30, 2026
Merged

feat: Added ability to lock a tactical map object to screen#3514
alexanderson1993 merged 1 commit into
developfrom
feat/add-stay-on-screen

Conversation

@mitchell-foote

Copy link
Copy Markdown
Contributor

Description

Implemented the ability to lock an tactical map icon to the screen borders.

As a note, there was a duplicated file, as the logic needed to be shared across the server and the client, and the tsconfig wasn't playing nice with trying to reference files across that barrier. If needed I can go back and do some additional modifications to force that to work, but overall, it should be pretty straightforward.

Code changes include:

Adding some height and width fields, as well as a boolean field to signify that it should lock to screen.

Additional code changes are about checking that field, and making sure that the icon stays where it needs to.

Related Issue

#3513

Screenshots (if appropriate):

Map:
Screenshot 2026-06-30 at 11 17 38 AM

Original Map after I've thrustered / AWSD / JKLI everything off the map:
Screenshot 2026-06-30 at 11 17 28 AM

New Map with icons that have the lock to screen boolean added:
Screenshot 2026-06-30 at 11 18 25 AM

  • I submitted a pull request or created an issue for documenting this
    feature on the Thorium Docs
    repo. (Include the issue or pull request url below.)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I could have sworn there was a way to share logic between the server and the client. Alas.

@alexanderson1993
alexanderson1993 merged commit 715f394 into develop Jun 30, 2026
1 check passed
@alexanderson1993
alexanderson1993 deleted the feat/add-stay-on-screen branch June 30, 2026 19:29
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