Skip to content

Conversation

iannesbitt
Copy link
Contributor

Closes #2376

@iannesbitt iannesbitt requested a review from robyngit April 19, 2024 23:07
--portal-grey-5: #80868B;
--portal-grey-6: #5F6368;
--portal-grey-7: #3C4043;
--portal-grey-8: #202124;
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi Ian! This color palette is specific to the light theme so I'd recommend not directly copying over the config from light.css. The issue described in the ticket is about layer category color, which is defined here. I think the fix should be two stepped:

  1. Set --portal-col-text-highlight in dark.css to a legible color in the context of dark theme. This variable is used by other components as well (.layer-details__label, .viewfinder-prediction__content) so you'll see a color change at those places. This fix will also only apply to portals that explicitly use the dark theme. Map in the main repo, for example, will still use the fallback color of --map-col-highlight__deprecate.

  2. To address the issue for non-theme users, we can replace the fallback color in map-view.css with the same color from # 1.

Copy link
Contributor Author

@iannesbitt iannesbitt Apr 29, 2024

Choose a reason for hiding this comment

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

Hi @yvonnesjy! Thank you, I appreciate the heads up and the detailed explanation. I just assumed that copying the blues and greys definitions to dark.css and changing the specific variable references would be sufficient to make the dark theme more legible without causing problems elsewhere. I can change to define them directly.

@robyngit robyngit added the revive needed outdated but valuable PRs & issues that are incomplete or need rethinking to align with codebase now label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

revive needed outdated but valuable PRs & issues that are incomplete or need rethinking to align with codebase now

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Colors in portal dark theme layer list don't quite fit after update

3 participants