Skip to content

feat: add CapeFlyer ocean blue color to FE#2640

Merged
robbie-sundstrom merged 5 commits intomainfrom
rs/add-capeflyer-blue-to-fe
Aug 11, 2025
Merged

feat: add CapeFlyer ocean blue color to FE#2640
robbie-sundstrom merged 5 commits intomainfrom
rs/add-capeflyer-blue-to-fe

Conversation

@robbie-sundstrom
Copy link
Contributor

Asana task: Add CapeFlyer ocean blue color to FE

Adding this new color to the Frontend will allow Outfront to deploy this new code so that we can use it on the backend.

  • Tests added?

@robbie-sundstrom robbie-sundstrom requested a review from a team as a code owner August 11, 2025 16:43
@digitalcora
Copy link
Contributor

I found some additional places we might want to add this, corresponding to mbta/screens-config-lib#68:

  • css/lcd_free_text
  • css/v2/dup/free_text
  • css/v2/pre_fare/free_text

For these, since we've added both capeflyer as a possible pill type and ocean_blue as a possible color, we'd want to include selectors for both of those (similar to how the existing definitions have both --cr and --purple).

@robbie-sundstrom
Copy link
Contributor Author

@digitalcora good call on including free_text changes here! Also added ferry selector to these files since it was missing, and bus for the pre_fare free_text since it was not currently included

@digitalcora
Copy link
Contributor

Nice catch on the additional missing pill types — that's one less surprise waiting for us later.

And, apologies, but I did think of one other thing: To have the new capeflyer pill type actually work in FreeText, it needs to be added to components/v2/free_text.tsx in iconPills and iconPaths. For the SVG path it would use the same one as cr since it's the same icon. (The ocean_blue color should work as-is since this is put directly into a classname and not filtered to a list of known colors or anything.)

Copy link
Contributor

@digitalcora digitalcora left a comment

Choose a reason for hiding this comment

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

🌊

@robbie-sundstrom robbie-sundstrom merged commit ccfc302 into main Aug 11, 2025
12 checks passed
@robbie-sundstrom robbie-sundstrom deleted the rs/add-capeflyer-blue-to-fe branch August 11, 2025 18:30
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.

2 participants