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

Baywalls Color-Correction Pass #5688

Merged
merged 22 commits into from
Jul 13, 2023

Conversation

AlphaM01
Copy link
Contributor

@AlphaM01 AlphaM01 commented Jul 9, 2023

About The Pull Request

This is a Code/Spriting PR. Some Map changes may follow in the future to address remaining issues,
The goal of the PR is to refine and consolidate color design choices of the new Baywalls. I've done what I can not to stray too far from theme where it's been voted on, though changes have been made to most departments. Rather, the aim is to execute the presentation more consistently.
Several colors previously at 100% saturation have been toned down, most department's palettes (except for Civ and Sci) have had tweaks and corrections made in some way. Atmospheric's chaotic color scheme in particular has been overhauled into a single proprietary department color.
Some new color defines have been added to avoid altering sprites which previously shared colors with certain departments.

Why It's Good For The Game

Much as Baywalls already look quite good, their current implementation has extremely clashing and oversaturated colors which look completely out of place as well as many inconsistencies.
These changes have been designed to retain the core principles behind those color choices while addressing their issues.

Seen as this PR is all about color, below is a(n outdated) visual summary of changes:

PR_Hallway
PR_Command
PR_Exploration
PR_Cargo
PR_Atmos
PR_Engineering
PR_Medical
PR_Security

Changelog

🆑
add: Defined three new colors: COLOR_SECURITY_RED, COLOR_ATMOSPHERICS_CYAN and COLOR_CARGO_BROWN;
add: Added "Maroon" mapping paint (Old Security color);
tweak: Altered COLOR_EXPLO_VIOLET, COLOR_SUN, COLOR_BABY_BLUE and COLOR_WALL_GUNMETAL;
tweak: Added stripes to Engineering airlocks;
tweak: Recolored stripes on Medical airlocks;
fix: Wall and door stripe sprites now match Airlock sprites in color.
/:cl:

New color define as Maroon was a pre-existing color already used elsewhere.
1st iteration SECURITY_RED still oversaturated in-game, mind that the def color is no longer the exact rendered color as the stripe overlay is no longer raw white.
Also changes a few things to use the new color which I'd missed the first time.
Sec darkred paint is now COLOR_SECURITY_RED instead of MAROON, used for windows.
Sec officer's closet also swapped away from MAROON.
darkred paint is the sec color, but just in case maroon paint has been added so it can still be used by mappers if they've a need for it.
Makes COLOR_BABY_BLUE hopefully match medbay tile overlays just that little better, very slight change overall.
Experimentally changes the medbay walls to be VERY_LIGHT_GRAY as GRAY80 is slightly too dark.
honing in on the correct color...
Experimentally colors Medbay door stripes BABY_BLUE instead of DEEP_SKY_BLUE to match wall stripes similar to most other departments.
As the airlocks are fairly dark gray despite being COLOR_WHITE.
SUN and AMBER look fine, investigating compatibility with the orange floor decals currently unused by Atlas.
Tiny COLOR_SUN change to better match tile overlay
Replaces atmospheric's salad of cyans with consolidated COLOR_ATMOSPHERICS_CYAN, actual value to be refined.
Some testing later...
50% cut in saturation
New color define as COLOR_BEASTY_BROWN is also used by several closets including Engineering which benefit from the extra saturation for readability.
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 9, 2023
now properly matches overlay
Copy link
Contributor

@Mazianni Mazianni left a comment

Choose a reason for hiding this comment

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

looks good codewise. see comments on discord re: color choices

@AlphaM01
Copy link
Contributor Author

Not doing all the fancy previews again, but it now Looks Better.
up-to-date IMGs in the Discord discussion.

@silicons silicons merged commit d492cbe into Citadel-Station-13:master Jul 13, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants