TrainTN Posted December 20, 2022 Posted December 20, 2022 Current IDs read like this: "ID (John Baldman (Assistant (NT)))". That's a lot of brackets stacking up on each other. If it is possible to do, I think it would read better if they were changed to this: "ID (John Baldman (NT Assistant))". Or if we wanted to cut down on as many brackets as possible: "ID (John Baldman, NT Assistant)". It's a small thing about presentation but I do think making things look nice where possible is important. 3
Zulu0009 Posted December 20, 2022 Posted December 20, 2022 This is a pet peeve I've had since forever and I never thought to make a post about. Three brackets hurt my soul, +1 for ID (John Baldman, NT Assistant) or ID (John Baldman - NT Assistant).
Carver Posted December 20, 2022 Posted December 20, 2022 This may look a little messier on the manifest when you're glancing out how many Physicians or Security Officers there are and you get something like: Dinkle, Idris Security Officer Poopy, PMCG Security Officer Feefoo, Zavod Security Officer Sure, it looks better on the ID itself, but the manifest is where this tends to matter most. Plus, some of the shortened corps like Zavod, Hepht and Zeng look frankly terrible when they're placed directly in front of a title. It only really works for the other half of corps who either have a full name, or use acronyms - overall it would need to be changed to use the acronym of every corp if it wanted to look uniform and neat, resulting in: Hepht = HI Idris = II (this is the only one that has potential to suffer aesthetically from being an acronym, but most fonts you can use rectify that) Orion = OE Zeng = ZHP Zavod = ZI and as an adjustment for uniformity, NT = NTC With everything being shortened and made into uniform acronyms, sure, I can see it working. In fact I would give it bonus points for making ZI synthetic names look even dumber, potentially freeing IPCs from that aesthetically ugly naming scheme.
Gem Posted December 20, 2022 Posted December 20, 2022 (edited) i'll look into this. Edited December 20, 2022 by Gem
Gem Posted December 20, 2022 Posted December 20, 2022 PR is up. https://github.com/Aurorastation/Aurora.3/pull/15392 1
TrainTN Posted December 20, 2022 Author Posted December 20, 2022 That looks good to me, definitely an improvement. Thank you.
Carver Posted December 21, 2022 Posted December 21, 2022 A very clean alternative in that PR, I like it!
Fluffy Posted November 9, 2023 Posted November 9, 2023 The PR has been merged and this suggestion is therefore considered implemented. Locking and archiving.
Recommended Posts