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

feat!: Make active Tab look less like a primary Button #1773

Merged
merged 6 commits into from
Dec 11, 2024

Conversation

VincentSmedinga
Copy link
Contributor

@VincentSmedinga VincentSmedinga commented Dec 4, 2024

Describe the pull request

What

Implements the updated design of Tabs.

Why

The active Tab Button looked too much like a primary Button, making the latter less prominent.

How

Tokens and CSS. I considered leaving some tokens in that we don’t need ourselves but could be useful for other communities to set background and border colors. However, I decided against it (see second commit) as we should first determine whether we want to adopt that approach at all. Even while it would make this change non-breaking.

They would be useful for communities that reuse our components and need background and border colours, but we should first consider whether we want to adopt this approach.
RubenSibon
RubenSibon previously approved these changes Dec 5, 2024
Copy link
Contributor

@RubenSibon RubenSibon left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot temporarily deployed to demo-DES-1018-update-tabs-design December 10, 2024 15:27 Destroyed
@github-actions github-actions bot temporarily deployed to demo-DES-1018-update-tabs-design December 10, 2024 15:31 Destroyed
packages/css/src/components/tabs/tabs.scss Outdated Show resolved Hide resolved
packages/react/src/Tabs/TabsButton.tsx Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to demo-DES-1018-update-tabs-design December 11, 2024 08:56 Destroyed
@github-actions github-actions bot temporarily deployed to demo-DES-1018-update-tabs-design December 11, 2024 08:59 Destroyed
@alimpens alimpens merged commit 2ec0102 into develop Dec 11, 2024
6 checks passed
@alimpens alimpens deleted the feature/DES-1018-update-tabs-design branch December 11, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants