Skip to content

Comments

fix(fuselage-tokens): incorrect color tokens on Button's success variant #1471

Merged
aleksandernsilva merged 3 commits intomainfrom
fix/button-colors
Oct 25, 2024
Merged

fix(fuselage-tokens): incorrect color tokens on Button's success variant #1471
aleksandernsilva merged 3 commits intomainfrom
fix/button-colors

Conversation

@aleksandernsilva
Copy link
Contributor

@aleksandernsilva aleksandernsilva commented Oct 23, 2024

Proposed changes (including videos or screenshots)

This PR fixes Button's success variant, which was using the incorrect color tokens. This was affecting the light theme in particular.

Before:
Screenshot 2024-10-23 at 16 32 05

After:
Screenshot 2024-10-23 at 16 33 07

Issue(s)

Further comments

VOIP-118

@changeset-bot
Copy link

changeset-bot bot commented Oct 23, 2024

🦋 Changeset detected

Latest commit: a4586be

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@rocket.chat/fuselage-tokens Patch
@rocket.chat/fuselage Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@aleksandernsilva aleksandernsilva marked this pull request as ready for review October 23, 2024 21:25
@aleksandernsilva aleksandernsilva merged commit 0b4e37a into main Oct 25, 2024
@aleksandernsilva aleksandernsilva deleted the fix/button-colors branch October 25, 2024 16:00
@github-actions github-actions bot mentioned this pull request Oct 28, 2024
dougfabris pushed a commit that referenced this pull request Oct 29, 2024
…ant (#1471)

Co-authored-by: Júlia Jaeger Foresti <60678893+juliajforesti@users.noreply.github.com>
dougfabris pushed a commit that referenced this pull request Oct 29, 2024
…ant (#1471)

Co-authored-by: Júlia Jaeger Foresti <60678893+juliajforesti@users.noreply.github.com>
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.

2 participants