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

Add relay map with no interactions as captains map #2175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Matombo
Copy link

@Matombo Matombo commented Sep 22, 2024

Gives the captains laptop something to display when he is not just providing the main screen. Arguably makes the captains role easyer as he needs to communicate less with relay.

This patch has a desgin bug that I'm not really sure how to fix: the new entry in alternative options sections in the ship station select screen does not fit the current layout. (and as such only the upper pixel row of the button is clickable). Any idea on how to make this look right again and make the button clickable in whole?

@aBlueShadow
Copy link
Contributor

aBlueShadow commented Sep 23, 2024

You would need to rearrange and resize the Gui elements in shipSelectionScreen.cpp (CrewPositionSelection)
That said, I am not sure if there is much of a benefit over using the strategic map and just not touching the buttons.

@Matombo
Copy link
Author

Matombo commented Sep 24, 2024

The benefit would be a cleaner look and another option for LARP settings.

Also "just not touching the buttons" might not work for every player group.

@daid
Copy link
Owner

daid commented Oct 2, 2024

I personally like it if the captain has zero direct interaction with the game.

The mainscreen can currently be switched to short and long range radar, maybe it makes sense to add the relay map as option there as well? And then have it auto-center on the player ship. So no interaction is required.

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.

3 participants