Skip to content

[dashboard] team setting design polish + more consistent color use #20547

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

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

filiptronicek
Copy link
Member

Description

Fixes some small nits in the organization settings pages and also adds our adaptive Podkit colors (pk-content-* and pk-surface-*) to improve readability - these colors adapt to both light and dark modes, so the resulting markup is way shorter and actually communicates intent, rather than an arbitrary color value (like text-gray-400).

Related Issue(s)

Fixes CLC-1097

How to test

Check it out on https://ft-color-fixes.preview.gitpod-dev.com/workspaces. There are some places with improved contrast (like the new organization button), but mostly changes are in code rather than actual design.

Here's the example of the new org button in dark mode:

Before After
image image

/hold

Copy link
Member Author

Choose a reason for hiding this comment

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

Changes here were done to align with our Figma doc for Podkit components

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

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

Thank for the polish! Could immediately spot a couple of places, and in dark-mode it's even more pronounced 💅

@filiptronicek filiptronicek self-assigned this Jan 24, 2025
@roboquat roboquat merged commit 58e1f55 into main Jan 24, 2025
17 of 18 checks passed
@roboquat roboquat deleted the ft/color-fixes branch January 24, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants