Skip to content

[kumo] Add background tint to secondary button hover state#185

Open
tccloudflare wants to merge 2 commits intocloudflare:mainfrom
tccloudflare:feat/secondary-button-hover
Open

[kumo] Add background tint to secondary button hover state#185
tccloudflare wants to merge 2 commits intocloudflare:mainfrom
tccloudflare:feat/secondary-button-hover

Conversation

@tccloudflare
Copy link

Added the color Kumo-tint to the secondary button hover state.

Copy link
Collaborator

@mattrothenberg mattrothenberg left a comment

Choose a reason for hiding this comment

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

Thank you! Can we add classes for the focus state, too?

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 5, 2026

npm i https://pkg.pr.new/@cloudflare/kumo@185

commit: fec1627

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Docs Preview

View docs preview

Commit: 74e9d0a

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Visual Regression Report

1 screenshot(s) with visual changes:

Button / Loading State

7 px (0%) changed

Before After Diff
Before After Diff
5 screenshot(s) unchanged
  • Button / Variants
  • Button / Sizes
  • Button / With Icon
  • Button / Icon Only
  • Button / Disabled State

Generated by Kumo Visual Regression

@tccloudflare tccloudflare force-pushed the feat/secondary-button-hover branch from 74e9d0a to 9217d6b Compare March 12, 2026 21:18
@tccloudflare
Copy link
Author

Thank you! Can we add classes for the focus state, too?

Yes, just added the update.

Copy link
Collaborator

@Brayden Brayden left a comment

Choose a reason for hiding this comment

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

Looking between the proposed changes here in the Docs Preview link versus what is live in kumo-ui.com but can't spot the difference on hover state.

I think the only change here (now) is the focus state on a not-disabled secondary button. Can we change the patch notes to reflect that by chance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants