⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@malinskibeniamin
Copy link
Contributor

Screenshot 2026-01-15 at 15 19 50 Screenshot 2026-01-15 at 15 19 56

@malinskibeniamin malinskibeniamin marked this pull request as ready for review January 15, 2026 18:33
Copy link
Contributor

@eblairmckee eblairmckee left a comment

Choose a reason for hiding this comment

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

Love this design so much more than current.

Comment on lines -55 to -61
const { colorMode } = useColorMode();

const isExpanded = isMobile || state === 'expanded';
const isDark = colorMode === 'dark';

const fullLogo = isDark ? RedpandaTextWhite : RedpandaTextColor;
const iconLogo = isDark ? RedpandaIconWhite : RedpandaIconColor;
Copy link
Contributor

Choose a reason for hiding this comment

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

making an assumption here that sidebar background color doesn't change enough in dark mode to warrant a different color for the brand name text?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly. Previously we had a redpanda orange colour text, now it's just dark

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly. Previously we had a redpanda orange colour text, now it's just dark

@eblairmckee eblairmckee merged commit d1d235e into master Jan 16, 2026
9 checks passed
@eblairmckee eblairmckee deleted the new-sidebar-assets branch January 16, 2026 03:28
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