Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds drawing to tacmaps #4475

Merged
merged 204 commits into from Nov 17, 2023
Merged

Adds drawing to tacmaps #4475

merged 204 commits into from Nov 17, 2023

Conversation

ghost
Copy link

@ghost ghost commented Sep 22, 2023

About the pull request

Features

  • Gives cic (sos, xo, co) and the queen the ability to draw on the tactical map and then announce it to their respective factions
  • All marine faction members now are able to access the tactical map announcement through the stat panel, which can be viewed at any time.
  • Xenos in addition to having the normal tactical map will also have access to their own canvas map that can be viewed when Queen is on ovi.
  • Ghosts can view the tactical maps for both factions in the Ghost tab as well as from ghost alert popups.
  • Admins can view all the round's drawings and optionally delete them.

Explain why it's good for the game

Hopefully this will spice things up for cic, queen and players alike. The idea is command and the queen would now be able to better convey what they want from their faction, which should help with general cohesion and likely expand strategic plays on both sides. (don't abuse it or harry will take it away, or make it more exclusive to the queen and co)

Testing photographs and procedure

Screen Shot 2023-09-23 at 6 38 25 PM Screen Shot 2023-09-23 at 6 39 08 PM Screen Shot 2023-09-23 at 6 40 22 PM Screen Shot 2023-09-23 at 6 42 54 PM

Changelog

🆑 Cthulhu80, Drathek
add: Adds drawing to tactical maps, viewable via stat panel for marines and xeno tacmap for xenos.
fix: Corrupted (and other hives) now have separate tactical maps.
/:cl:

Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

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

just this

code/modules/admin/tacmap_panel/tacmap_admin_panel_tgui.dm Outdated Show resolved Hide resolved
code/modules/mob/new_player/new_player.dm Outdated Show resolved Hide resolved
@harryob harryob added this pull request to the merge queue Nov 17, 2023
Merged via the queue into cmss13-devs:master with commit 86f2489 Nov 17, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Testmerge Candidate we'll test this while you're asleep and the server has 10 players UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pred's tacmap is broken tacmap does not care what hive you're part of
8 participants