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

Movistar new border radius #1961

Open
wants to merge 1 commit into
base: production
Choose a base branch
from
Open

Conversation

yceballost
Copy link
Collaborator

@yceballost yceballost commented Nov 18, 2024

Pull Request Description: Movistar New Border Radius

This pull request introduces updated border radius values for various components in the Movistar design tokens. The changes include:

  • Button: Updated from 4 to 999 to create a fully rounded appearance.
  • Checkbox: Increased from 2 to 4 for a softer look.
  • Container: Adjusted from 8 to 16 to enhance the visual separation of elements.
  • Input: Changed from 8 to 12 for improved aesthetics.
  • Sheet: Updated from 8 to 16 for consistency with the container.

Motivation

The primary motivation behind these changes is to create a more cohesive and modern visual identity across the Movistar components. By standardizing the border radius values, we enhance the user experience by making elements feel more integrated and approachable. The new values promote a softer, more inviting design language, which is crucial for user engagement and satisfaction.

Impact

These updates will improve the overall look and feel of the interface, ensuring that the design aligns with contemporary UI trends. The changes are expected to provide a more polished and professional appearance, reinforcing brand identity while maintaining usability.

@yceballost yceballost requested a review from aweell November 18, 2024 16:27
@yceballost yceballost marked this pull request as ready for review November 18, 2024 16:27
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.

2 participants