Skip to content

Add active state to Button component#7

Merged
Mike Mitchell (m10l) merged 1 commit intomasterfrom
feature/button-active-state
Nov 22, 2018
Merged

Add active state to Button component#7
Mike Mitchell (m10l) merged 1 commit intomasterfrom
feature/button-active-state

Conversation

@m10l
Copy link
Copy Markdown
Contributor

This change adds active states to Button components.

Some changes needed to be made to the transitions and the way we had defined backgrounds - you can't animate a gradient to a solid background as they're two different properties - background-image and background-color.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@m10l Mike Mitchell (m10l) merged commit 7e69b05 into master Nov 22, 2018
@m10l Mike Mitchell (m10l) deleted the feature/button-active-state branch November 22, 2018 16:05
Johannes Bugiel (wichniowski) pushed a commit that referenced this pull request Dec 10, 2018
toniwowtscherk added a commit that referenced this pull request Apr 29, 2026
Adds screenshot comparison post-flight for Figma-to-code (Path C):
capture source Figma design, render generated code, compare both,
fix differences until converging. Eval showed V3 Path C missed sidebar,
top nav, filter chips, and avatar columns because the design context
alone doesn't describe page-level composition.

Also syncs eval-driven fixes from product-design PR #7: corrected icon
names (TrashIcon not TrashSimpleIcon), removed fabricated Skeleton.Row,
fixed shadow token names.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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