-
Notifications
You must be signed in to change notification settings - Fork 619
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
Civillian and Medical ship Holopad update #2800
Conversation
There is a PR out for the Comet #2690 with a holopad already added. Does that seem acceptable? To prevent a conflict. |
Is any maptainer testing this or I'll need to? 👀 |
I'll get on with testing it tomorrow. :> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've gone through all 42 ships, and the changes are pretty minimal. Looking promising so far!
Comments:
- Bulker: I don't love the placement of the holopad or the station records computer, but I will approve both. I'm not sure how else one could reasonably arrange things within the extremely limited available space.
- Comet: There is an open PR to revamp the Comet (Comet Defib Update #2690). I will merge this one first, as the other PR is enough of a rework that it'll probably take some time to review it properly, and I'd like to get holopads sorted sooner rather than later.
- Crescent, Harbormaster, Loader, Lyrae: The holopad is on top of a vent. Depending seemingly on the game's whims, though the holopad is visually on top, sometimes the vent "steals" mouse focus, forcing you to carefully click on the part of the holopad that's outside the vent. It's not great. I'm prepared to allow it for now, but I'd like to find a different, better long-term solution.
- McDelivery: This seems to have lost its station records computer. Are we no longer going to require those on tiny daughterships?
Ships not touched by this PR:
- Akupara
- Construct
Please verify that these are intended to be excluded. :)
Record computer needed to track owner |
Are you sure about that? 🤔 |
I've recently added a holopad to the Bodkin on my open PR to update it #2640 |
That does seem bad. I understand maybe the harbormaster and Loader not having enough space to put the holopad, but is there really not enough space on the lyrae and the crescent (with crescent being one of the biggest ships currently in the game, and lyrae having that pretty large open lobby/hallway)? |
I mean you see the ship owner on it |
I mean, only half of the time iirc but thats another issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outstanding issues resolved in conversation on Discord. Over-vent holopads will be accepted for now. Akupara has a PR that reworks it (#2816), will let that introduce the holopad. Construct is skipped for now, will be included in a later batch. All ships approved. Let's get this show on the road. :)
About the PR
Adds holopads to the 42 civilian/medical ships. Missing are, Caduceus(pending replacement any day now), Barge (already implemeneted), Pioneer(changes incoming) and Camper (no room-need design consideration)
Why / Balance
#2795 (comment)
How to test
Spawn the ships in your favorite method, in game or dev or mapping or testing, and see the glorious holopads
Media
Requirements
Breaking changes
Changelog
🆑