Skip to content

Conversation

@jvsena42
Copy link
Member

@jvsena42 jvsena42 commented Dec 3, 2025

FIGMA

Description

This PR update the Primary, Secondary and Tertiary Buttons to v59. The other components. will be updated in other PR

Preview

Figma Android

QA Notes

@jvsena42 jvsena42 self-assigned this Dec 3, 2025
@jvsena42 jvsena42 mentioned this pull request Dec 3, 2025
@jvsena42 jvsena42 changed the title Feat/buttons v59 Buttons v59 Dec 4, 2025
@jvsena42 jvsena42 marked this pull request as ready for review December 4, 2025 12:45
@jvsena42 jvsena42 requested a review from Copilot December 4, 2025 12:46
@jvsena42 jvsena42 changed the title Buttons v59 Primary, Secondary and Tertiary Buttons v59 Dec 4, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the button styling to match the v59 design specifications from Figma. The changes introduce a new gradient-based visual style for primary buttons with shadow effects and refined color values across the theme.

Key changes:

  • Updated gray color palette values for improved visual consistency
  • Implemented new primary button styling with gradient backgrounds and shadow effects
  • Replaced alpha-based disabled states with ColorFilter.tint for better color consistency

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
Colors.kt Updated gray color palette values and added Gray7 to support new design system
Modifiers.kt Added primaryButtonStyle modifier with gradient backgrounds, shadow, and border effects
Button.kt Refactored button components to use new styling system with ColorFilter instead of alpha for disabled states
ShowMnemonicScreen.kt Removed hardcoded color parameter to use default button styling

@jvsena42 jvsena42 marked this pull request as draft December 4, 2025 12:51
@jvsena42 jvsena42 marked this pull request as ready for review December 4, 2025 14:07
@jvsena42 jvsena42 enabled auto-merge December 4, 2025 14:12
@jvsena42 jvsena42 requested a review from ovitrif December 4, 2025 14:13
@jvsena42 jvsena42 requested a review from ben-kaufman December 4, 2025 14:13
@jvsena42 jvsena42 mentioned this pull request Dec 4, 2025
5 tasks
Copy link
Collaborator

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

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

tAck

@jvsena42 jvsena42 merged commit 25b2a4e into master Dec 4, 2025
12 of 13 checks passed
@jvsena42 jvsena42 deleted the feat/buttons-v59 branch December 4, 2025 21:42
@ovitrif
Copy link
Collaborator

ovitrif commented Dec 8, 2025

Fond this bug:

Main Buttons are not always Full-Width

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